Closed duongnt closed 4 years ago
Merging #42 into master will decrease coverage by
0.53%
. The diff coverage is25%
.
@@ Coverage Diff @@
## master #42 +/- ##
==========================================
- Coverage 91.54% 91.01% -0.54%
==========================================
Files 2 2
Lines 343 345 +2
==========================================
Hits 314 314
- Misses 29 31 +2
Impacted Files | Coverage Δ | |
---|---|---|
samlauthenticator/samlauthenticator.py | 90.98% <25%> (-0.54%) |
:arrow_down: |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update b319e77...26c8cee. Read the comment docs.
I think this is a good start, but I'm a little concerned that we're breaking backward compatibility with this change. Someone out there who has slo_forwad_on_logout
set might get broken by this. Is it ok if I make a PR against your branch?
I think this is a good start, but I'm a little concerned that we're breaking backward compatibility with this change. Someone out there who has
slo_forwad_on_logout
set might get broken by this. Is it ok if I make a PR against your branch?
Sure. I guess we can leave the typo slo_forwad_on_logout
there for backward-compatibility, but add a [Deprecated] note?
@duongnt If you merge my distortedsignal:ds-minor-changed
branch into your branch, I'll merge your branch into this master and then cut a bugfix release.
@duongnt If you merge my
distortedsignal:ds-minor-changed
branch into your branch, I'll merge your branch into this master and then cut a bugfix release.
Thanks @distortedsignal, I merged your changes to my master.
Hello, Thanks for awesome work! The SAMLAuthenticator is mostly working as documented. Ran into an issue when setting the c.SAMLAuthenticator.slo_forwad_on_logout = True . Tried both 'forward' and 'forwad'. What should this value be set to?
[E 2020-03-16 18:36:36.179 JupyterHub web:1670] Uncaught exception GET /hub/logout (::ffff:10.x.x.x) HTTPServerRequest(protocol='http', host='myhost.com:8000', method='GET', uri='/hub/logout', version='HTTP/1.1', remote_ip='::ffff:10.x.x.x') Traceback (most recent call last): File "/opt/anaconda3/lib/python3.7/site-packages/tornado/web.py", line 1592, in _execute result = yield result File "/opt/anaconda3/lib/python3.7/site-packages/samlauthenticator/samlauthenticator.py", line 754, in get forwad_on_logout = True if authenticator_self.slo_forwad_on_logout else False AttributeError: 'SAMLAuthenticator' object has no attribute 'slo_forwad_on_logout'
Fix forwad -> forward (Fix #40)
Signed-off-by: Thi Duong Nguyen