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

"authorization_header" field should be clarified and added to the "Stream Configuration" section (7.1.1) #223

Open
tulshi opened this issue Dec 6, 2024 · 1 comment
Open

"authorization_header" field should be clarified and added to the "Stream Configuration" section (7.1.1) #223

tulshi opened this issue Dec 6, 2024 · 1 comment
Labels
spec:SSF v1Final Issues that must be fixed before we propose a spec to become a v1 final spec.

Comments

@tulshi
Copy link
Contributor

tulshi commented Dec 6, 2024

The spec adds an "authorization_header" field in section 10.3.1.1 (SET Token Delivery Using HTTP Profile -> Stream Configuration Metadata -> Push Delivery Using HTTP). The description of this field is also ambiguous in that the words "if the configuration is present" doesn't clarify which configuration this is.

This language (in fact the whole section 10.3.1) should be reconciled with the Stream Configuration section.
@tulshi tulshi mentioned this issue Dec 6, 2024
Closed
@FragLegs
Copy link
Contributor

We will consider this a subtask of #226

@FragLegs FragLegs added v1Final Issues that must be fixed before we propose a spec to become a v1 final spec. spec:SSF labels Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
spec:SSF v1Final Issues that must be fixed before we propose a spec to become a v1 final spec.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@FragLegs @tulshi