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.

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

Module plone.session.tktauth, line 247, in splitTicket #33

Open
zopyx opened this issue Oct 9, 2022 · 7 comments
Open

Module plone.session.tktauth, line 247, in splitTicket #33

zopyx opened this issue Oct 9, 2022 · 7 comments

Comments

@zopyx
Copy link
Member

zopyx commented Oct 9, 2022

Plone 6.0.0b3, plone.session 4.0.0b2

I receive this error for each page load on a fresh Plone site with content added through collective.exportimport:

2022-10-10 12:45:29,366 ERROR   [Zope.SiteErrorLog:35][waitress-0] ValueError: http://dev2.zopyx.de:43080/eteaching/++webresource++6e394ff2-5e80-5880-9df7-ed8e59129465/acl_users/session/refresh
Traceback (innermost last):
  Module ZPublisher.WSGIPublisher, line 167, in transaction_pubevents
  Module ZPublisher.WSGIPublisher, line 376, in publish_module
  Module ZPublisher.WSGIPublisher, line 271, in publish
  Module ZPublisher.mapply, line 85, in mapply
  Module ZPublisher.WSGIPublisher, line 68, in call_object
  Module plone.session.plugins.session, line 432, in refresh
  Module plone.session.plugins.session, line 403, in _refreshSession
  Module plone.session.plugins.session, line 231, in _validateTicket
  Module plone.session.tktauth, line 246, in splitTicket
ValueError
ticket = b'61646d696e:61646d696e'
encoding = None
@zopyx zopyx added this to the Plone 6.0 milestone Oct 9, 2022
@gforcada
Copy link
Member

gforcada commented Oct 9, 2022

That line is trying to convert the ticket to a number, there is a colon on the ticket you posted, so no wonder it breaks, why there is a colon though? 🤔

@zopyx
Copy link
Member Author

zopyx commented Oct 9, 2022

No idea...I doubt that is coming from out policy file...I do not even know what tkauth is supposed to do 😇

@mauritsvanrees
Copy link
Member

Can you post the full traceback please? It does not include an error.

@zopyx
Copy link
Member Author

zopyx commented Oct 10, 2022

Can you post the full traceback please? It does not include an error.

I updated the original traceback

@gforcada
Copy link
Member

I see that @ale-rt removed six before 4.0.0b2 was released. File history and the specific commit.

Would you happen to have time to test if it happens with 4.0.0b1 ? maybe on that six removal is the problem, given that it raises a ValueError and your input seems to be bytes ?

All of this is wild guessing at 5:40 at night, so take it with a bit of🧂

@ale-rt
Copy link
Member

ale-rt commented Nov 30, 2022

The ticket is simply too short... it should be 32 chars (which is a checksum) + 8 chars (a timestamp in hex) + whatever.
No idea why you ticket is so short and in the form $16_digit_hex:$16_digit_hex

@gogobd
Copy link

gogobd commented Feb 10, 2025

We get the same ValueError.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants