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.

Sign up for GitHub

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

Jump to bottom

chore: modify the description of ssl.key_encrypt_salt in config-default.yaml #10220

Merged
merged 1 commit into from
Sep 19, 2023
Merged

chore: modify the description of ssl.key_encrypt_salt in config-default.yaml #10220

merged 1 commit into from
Sep 19, 2023

Conversation

Sn0rt
Copy link
Contributor

@Sn0rt Sn0rt commented Sep 18, 2023

Description

Fixes #10206

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

@Sn0rt
fix: ssl.key_encrypt_salt field description
06ca921 
Signed-off-by: Sn0rt <wangguohao.2009@gmail.com>
@monkeyDluffy6017 monkeyDluffy6017 changed the title fix: ssl.key_encrypt_salt field description chore: ssl.key_encrypt_salt field description Sep 18, 2023
@monkeyDluffy6017 monkeyDluffy6017 changed the title chore: ssl.key_encrypt_salt field description chore: modify the description of ssl.key_encrypt_salt in config-default.yaml Sep 19, 2023
@monkeyDluffy6017 monkeyDluffy6017 merged commit 9b2031a into apache:master Sep 19, 2023
@Sn0rt Sn0rt deleted the api7/config-file-comment branch September 20, 2023 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kayx23 kayx23 kayx23 approved these changes

@monkeyDluffy6017 monkeyDluffy6017 monkeyDluffy6017 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

docs: ssl.key_encrypt_salt The comments for this field are unclear.
3 participants
@Sn0rt @monkeyDluffy6017 @kayx23