Skip to content
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

Update pki tps-config doc #3643

Merged
merged 3 commits into from
Jul 8, 2021
Merged

Update pki tps-config doc #3643

merged 3 commits into from
Jul 8, 2021

Conversation

edewata
Copy link
Contributor

@edewata edewata commented Jul 7, 2021

The doc for pki tps-config has been updated to use JSON instead of XML file format:
https://github.com/edewata/pki/blob/json/docs/changes/v11.0.0/Switching-from-XML-to-JSON-REST-API.adoc
https://github.com/edewata/pki/blob/json/docs/user/tools/Using-PKI-TPS-Configuration-CLI.adoc

The Link constructor has been modified not to set the type if it's not specified to match RESTEasy's Link constructor:
https://github.com/resteasy/Resteasy/blob/3.0.26.Final/providers/resteasy-atom/src/main/java/org/jboss/resteasy/plugins/providers/atom/Link.java#L54-L58

Note: This PR should be merged together with PR #3610.

The Link constructor has been modified not to set the type
if it's not specified to match RESTEasy's Link constructor:
https://github.com/resteasy/Resteasy/blob/3.0.26.Final/providers/resteasy-atom/src/main/java/org/jboss/resteasy/plugins/providers/atom/Link.java#L54-L58
The doc for pki tps-config has been updated to use
JSON instead of XML file format.
@ckelleyRH
Copy link
Contributor

@edewata this WARNING: Unable to get the value of cipher CI failure is really starting to annoy me, I've re-run the workflow.

Copy link
Contributor

@ckelleyRH ckelleyRH left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for doing this!

@edewata
Copy link
Contributor Author

edewata commented Jul 8, 2021

@edewata this WARNING: Unable to get the value of cipher CI failure is really starting to annoy me, I've re-run the workflow.

I filed these tickets for that issue:
dogtagpki/jss#781
dogtagpki/jss#782

The code is quite low-level and complex, so I'm not sure who will be able to fix it properly, but it seems to be happening only when closing the connection, so we might be able to just add a code to ignore the NPE.

@edewata
Copy link
Contributor Author

edewata commented Jul 8, 2021

@ckelleyRH Thanks! I'll merge both PRs.

@edewata edewata merged commit a51f61a into dogtagpki:master Jul 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants