[irods/irods#2626] Update SSL documentation (main)#326
Merged
alanking merged 2 commits intoirods:mainfrom Apr 1, 2025
Merged
Conversation
korydraughn
reviewed
Mar 31, 2025
Collaborator
korydraughn
left a comment
korydraughn
left a comment
There was a problem hiding this comment.
Looks good overall.
korydraughn
approved these changes
Apr 1, 2025
Collaborator
korydraughn
left a comment
korydraughn
left a comment
There was a problem hiding this comment.
Squash and pound it.
Collaborator
|
Don't forget to check the commit message to make sure it aligns with recent changes. |
Contributor
Author
|
Squashed. Will # after discussing some clarifying questions... |
Contributor
Author
|
We've decided to do the SSL / TLS swap, so I'll go ahead and do that in this PR. Plus, I noticed another reference to federating with 3.x in the Troubleshooting docs, so I'm going to remove that as well. |
Contributor
Author
|
Okay, I did the SSL -> TLS transition in the latest commit. I'm reserving some other edits for another PR based on these changes. |
korydraughn
reviewed
Apr 1, 2025
Contributor
Author
|
Squashed |
This commit makes the following changes to the SSL documentation: - Rename SSL page SSL/TLS - Removes explanation about using encryption only for data or only for passwords on TLS page - Move SSL configuration instructions out of PAM page to SSL/TLS page - Move some TLS configurations which were previously in the client configuration into the server configuration - More cleanly separated client and server TLS configuration sections The wording remains largely in tact with the exception of sections which dealt directly with the configurations which changed.
Contributor
Author
|
#'d, mergin |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In service of irods/irods#2626
The wording really is mostly the same. I'm willing to update parts if desired, but I tried to leave it as in-tact as possible, with "Configure iRODS server for TLS" being the notable exception.