-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Option to filter APIs based on their http-security #4
Comments
Solution 1 - filter by a PredicateAdd a Predicate was introduced in Java 8. Perhaps we should use an alternative class to stay compatible with Java 7. This method would allow client implementers to include any kind of filter. However, it's not very easy to use, because implementers will need to implement Predicates by themselves. Perhaps we can do better than that. |
Solution 2 - setRequiredHttpSecurityOptionsWe could also add a
Perhaps we should name this method a bit differently, in order to make it obvious that not all APIs declare these security options. For example, |
If we choose solution 2 though, then we will need to update the Registry Client every time a change occurs in any of the Unfortunately, I am leaving for vacation now, so I won't have time to solve this issue in the next weeks. For now, implementers should implement the filtering by themselves. |
I'm not sure how much "wanted" this feature is. Should I try to implement it before a leave the project? |
As of 2022, we have just one security option. I think this issue can be closed as 'Won't fix'. |
Previously, we wrote (here) that Java Registry Client won't help developers with filtering APIs based on their
<http-security>
. Quote:However, I am not certain about this. Perhaps we can bend the rules a bit, and include some v2-security-specific code in the common Registry Client. After all, we expect that most APIs will follow v2-security, so it would make sense to allow
ApiSearchConditions
to filter on them.Not sure how the new methods should look like though.
The text was updated successfully, but these errors were encountered: