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
Clarify documentation of SKIP_VERIFY #12203
Clarify documentation of SKIP_VERIFY #12203
Conversation
e847f79
to
9139bda
Compare
Please also sync to here. |
The documentation clearly documents the empty value as the default, however at least one user reported this as being unclear. Mark values explicitly so it is clear what values it can take. This clarifies that an empty value in fact leaves certificate verification enabled, whereas it has to be explicitly set to true to disable certificate verification. Resolves: go-gitea#12117 Signed-off-by: Alexander Scheel <alexander.m.scheel@gmail.com>
9139bda
to
df0e7d9
Compare
ACK, done. Thanks @silverwind. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should only use true/false for boolean examples and not mention empty. While empty will always evaluate to false, having it as a 3rd option seems to be what caused the original confusion this is meant to fix. Showing false as the default makes that more clear. Also slight change to comment just so it is the same between both files
Co-authored-by: mrsdizzie <info@mrsdizzie.com>
Co-authored-by: mrsdizzie <info@mrsdizzie.com>
Co-authored-by: mrsdizzie <info@mrsdizzie.com>
Sorry, was out of internet range. Updated! |
The documentation clearly documents the empty value as the default,
however at least one user reported this as being unclear. Mark values
explicitly so it is clear what values it can take. This clarifies that
an empty value in fact leaves certificate verification enabled, whereas
it has to be explicitly set to true to disable certificate verification.
Resolves: #12117
Signed-off-by: Alexander Scheel <alexander.m.scheel@gmail.com>