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

auth_data.yaml: formally allow it to be non-object #1115

Merged
merged 2 commits into from Jun 14, 2022

Conversation

KitsuneRal
Copy link
Member

@KitsuneRal KitsuneRal commented Jun 12, 2022

Closes #716.

Signed-off-by: Alexey Rusakov Kitsune-Ral@users.sf.net

Preview: https://pr1115--matrix-spec-previews.netlify.app

@KitsuneRal KitsuneRal requested a review from a team as a code owner June 12, 2022 11:46
@richvdh richvdh merged commit cbdd889 into main Jun 14, 2022
@richvdh richvdh deleted the kitsune/auth-data-definition-fix branch June 14, 2022 15:35
KitsuneRal added a commit to quotient-im/gtad that referenced this pull request May 3, 2024
analyzeTypeUsage() incorrectly coerces an empty type to `object` - see
matrix-org/matrix-spec#1115 for the case that
breaks things (even with this PR merged the generated type remained
`string->object` instead of `string->`).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Authentication data should be a map of strings to anything, not to objects
3 participants