syncope-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dmitriy333 <>
Subject [GitHub] syncope pull request #:
Date Wed, 04 Apr 2018 11:18:48 GMT
Github user Dmitriy333 commented on the pull request:
    In common/lib/src/main/java/org/apache/syncope/common/lib/search/
    In common/lib/src/main/java/org/apache/syncope/common/lib/search/
on line 39:
    @ilgrosso , Seems, that it would be great to make `token` field searchable again.
    Reset password notification may contain "about" value with a "filter" property.
    And "token" field of a "UserTO" may be used in a fiql query of password 
    reset notification. 
    E.g. `token!=$null` query can be used as a filter value JPAAnyAbout entity which is a
part of  JPANotification entity.
    If token field is not searchable for UserTO object, then SearchCondVisitor#createAttributeCond(final
String schema) creates AttributeCond object instead of AnyCond object, and therefore NotificationManagerImpl
skips password reset notification and doesn't process it correctly (look at line 344 in
    FYI, I'm using release version 2.0.8


View raw message