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.

Sign up for GitHub

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

Jump to bottom

Tuple syntax no longer allows / to be used #222

Open
annevk opened this issue Sep 29, 2023 · 2 comments
Open

Tuple syntax no longer allows / to be used #222

annevk opened this issue Sep 29, 2023 · 2 comments

Comments

@annevk
Copy link
Collaborator

annevk commented Sep 29, 2023

Infra dropped this quite a while ago, but it seems to be still used here for cookie attributes?
@inexorabletash
Copy link
Member

We must have missed a memo. 😉

This can easily be changed (PRs welcome?); ultimately this is populating a cookie-attribute-list which is a list of attributes, each one with an attribute-name and attribute-value. We approximate that with a list of lists, list of tuples, ordered map of key/value, define new concept types, etc. I don't have a strong preference.

@annevk
Copy link
Collaborator Author

annevk commented Sep 29, 2023

Yeah, I'm not immediately PR'ing as Johann and I are also looking at changes needed to 6265bis so the contract might well change a bit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@inexorabletash @annevk