mirror of https://github.com/OpenIdentityPlatform/OpenDJ.git

Guy Elsmore-Paddock
26.46.2017 40be863aec75ae0c0c7b97ad5622ff47f476a783
refs
author Guy Elsmore-Paddock <guy@rosieapp.com>
Thursday, October 26, 2017 03:46 +0200
committer Guy Elsmore-Paddock <guy@rosieapp.com>
Thursday, October 26, 2017 03:46 +0200
commit40be863aec75ae0c0c7b97ad5622ff47f476a783
tree 49c42bcba2c2f93bd8c08f542bbf330c25b5c0c0 tree | zip | gz
parent a2f838c8ea5c73db9651fec9cdf9d71a60efda06 view | diff
Fix read-only sub-resource CREST API descr (#6)

Corrects three obscure defects in the Rest2LDAP implementation of CREST descriptors:
- Read-only sub-resources were not appearing at all in the CREST API description JSON.
- When read-only sub-resources appeared in the API description alongside writable sub-resources for the same models, the generated service name for the sub-resources was the same, leading to an IllegalStateException. Now, we generate a unique service name for each sub-resource based on its writability.
- A top-level `create` request was still being rendered in the API description for read-only sub-resources.
5 files modified
198 ■■■■ changed files
opendj-rest2ldap-servlet/src/main/webapp/WEB-INF/classes/rest2ldap/endpoints/api/example-v1.json 12 ●●●●● diff | view | raw | blame | history
opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/ReadOnlyRequestHandler.java 13 ●●●●● diff | view | raw | blame | history
opendj-rest2ldap/src/main/java/org/forgerock/opendj/rest2ldap/Resource.java 57 ●●●● diff | view | raw | blame | history
opendj-rest2ldap/src/test/java/org/forgerock/opendj/rest2ldap/Rest2LdapJsonConfiguratorTest.java 104 ●●●● diff | view | raw | blame | history
opendj-server-legacy/resource/config/rest2ldap/endpoints/api/example-v1.json 12 ●●●●● diff | view | raw | blame | history