-
Notifications
You must be signed in to change notification settings - Fork 76
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
let people login to zuzalu.city with either a new ticket or old ticket either today or tomorrow #619
Comments
So what Zupass did was this: Users can provide a semaphore signature PCD, where the signed message is the user's UUID from the This worked because the only way to become a user in Zupass was to have a Zuzalu ticket, so there was no need for the PCD to make any claims about being a ticket-holder - transmitting the UUID was enough, because having a UUID meant having a ticket. PCDPass does support the Zuzalu fetchParticipant/fetchUser API, and with some tweaks can be made to work in the same way that the Zupass one did. Since this API will only work for Zuzalu ticket-holders, everything is fine. However, none of this works for Zuconnect tickets, for the reason that the PCDPass server doesn't know anything about Zuconnect tickets. I think the title of this ticket is a bit ambiguous: I originally interpreted "new ticket" to mean a Zuconnect ticket, and "old ticket" to mean a Zuzalu 1.0 ticket ported to PCDPass (as per #618). But now I wonder if "new tickets" are just the old tickets ported to PCDPass, and Zuconnect tickets are another issue altogether. |
I think both Zuconnect and Zuzalu tickets should be issued as new This may require you to augment the |
I don't think it needs to ask for multiple, it just needs one. If we imagine someone who has a Zuzalu1 ticket, a Zuconnect ticket, and one or more Devconnect tickets, then when they're prompted to select the PCD they will have to choose which one to provide. If they choose a non-Zuzalu/Zuconnect ticket then it shouldn't work (zuzalu.city should reject it). |
The hard part here was figuring out what data zuzalu.city really uses, and whether it's therefore safe to ignore some of the data that is currently being provided by the Zuzalu API (e.g. the resident/visitor/organizer role - although zuzalu.city stores this, it doesn't actually use it for anything, so it's OK that the PCD doesn't include it). |
Remaining to do is:
|
@robknight could you send me the PR in which you implemented this on the zuzalu.city repository please |
this already works |
No description provided.
The text was updated successfully, but these errors were encountered: