Skip to content

Refine TLS topic#402

Merged
zuiderkwast merged 4 commits intovalkey-io:mainfrom
yang-z-o:tls
Feb 3, 2026
Merged

Refine TLS topic#402
zuiderkwast merged 4 commits intovalkey-io:mainfrom
yang-z-o:tls

Conversation

@yang-z-o
Copy link
Contributor

@yang-z-o yang-z-o commented Jan 26, 2026

Changes include:

Signed-off-by: Yang Zhao <zymy701@gmail.com>
Signed-off-by: Yang Zhao <zymy701@gmail.com>
Signed-off-by: Yang Zhao <zymy701@gmail.com>
Signed-off-by: Yang Zhao <zymy701@gmail.com>
Copy link
Contributor

Choose a reason for hiding this comment

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

Renaming this file is a good idea, but we will need to create redirects in the website repo for the old URL. @stockholmux FYI.

Copy link
Member

@stockholmux stockholmux Jan 26, 2026

Choose a reason for hiding this comment

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

So https://valkey.io/topics/encryption/ will forward to https://valkey.io/topics/tls/?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I’ve opened a PR for the URL forwarding here: valkey-io/valkey-io.github.io#450
Please let me know if this is the correct way to handle it.

@zuiderkwast zuiderkwast self-requested a review January 26, 2026 09:58
zuiderkwast pushed a commit to valkey-io/valkey-io.github.io that referenced this pull request Feb 3, 2026
### Description

Renamed doc file `topics/encryption.md -> topics/tls.md`, need to
forward the old URL to new one.
Can wait until valkey-io/valkey-doc#402 is
merged.
 
### Issues Resolved

N/A

### Check List
- [X] Commits are signed per the DCO using `--signoff`

By submitting this pull request, I confirm that my contribution is made
under the terms of the BSD-3-Clause License.

Signed-off-by: Yang Zhao <zymy701@gmail.com>
@zuiderkwast zuiderkwast merged commit 4ba6ffe into valkey-io:main Feb 3, 2026
5 checks passed
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.

3 participants