-
Notifications
You must be signed in to change notification settings - Fork 669
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
NOISSUE - Fix typo in authorization.js #1226
Conversation
Signed-off-by: Ivan Milošević <iva@blokovi.com>
@@ -5,7 +5,7 @@ function authenticate(s) { | |||
if (!s.variables.ssl_client_s_dn || !s.variables.ssl_client_s_dn.length || | |||
!s.variables.ssl_client_verify || s.variables.ssl_client_verify != "SUCCESS") { | |||
s.deny(); | |||
return | |||
return; |
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.
can you do the same in UI repo
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.
Sure. I'll change it in devops also, that's where I spot it
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.
Here it is
I couldn't even start nginx without this fix in kubernetes because of this syntax error, but obviously in other environments this is not an issue.
Codecov Report
@@ Coverage Diff @@
## master #1226 +/- ##
=======================================
Coverage 76.39% 76.39%
=======================================
Files 106 106
Lines 6964 6964
=======================================
Hits 5320 5320
Misses 1259 1259
Partials 385 385 Continue to review full report at Codecov.
|
Signed-off-by: Ivan Milošević <iva@blokovi.com>
Signed-off-by: Ivan Milošević iva@blokovi.com
Pull request title should be
MF-XXX - description
orNOISSUE - description
where XXX is ID of issue that this PR relate to.Please review the CONTRIBUTING.md file for detailed contributing guidelines.
What does this do?
Add missing semicolon in
authorization.js