Not sure if this is a bug, or functionality that hasn't been implemented yet-

Trying to add attributes to the SAML response, created a class that implements AttributeManager and pulls data from our own in-house person manager tool (backend is LDAP, database and other authoritative sources). It seems to work properly but when it tries to build the SAML response I get :

at org.picketlink.identity.federation.core.saml.v2.util.StatementUtil.createAttributeStatement(StatementUtil.java:139)

From what I can decern it looks like its trying to convert the attrbitue names into a standard naming convention. But in StatementUtil only 4 of the dozen or so attributes in AttributeConstants seem to be implemented, and I don't see that anything has been implemented to handle the attributes that don't fall under the X500 spec.

In the picketlink-idfed.xml file there is currently the ServiceURL which defines where the IDP will send the response back to. Since picketlink intercepts any request made to a protected url on the SP are there any plans to update this to grab the current url before forwarding to the IDP that way this doesn't need to be hardcoded in there. Just thinking of instances where a user has a page bookmarked. They go to that page, get redirected to the IDP, then IDP redirects back to the home page of the site because that was what was specified as the ServiceURL.