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

[CHANGE REQUEST] Remove request header field "API" #529

Open
matt-wirtz opened this issue Feb 7, 2024 · 2 comments
Open

[CHANGE REQUEST] Remove request header field "API" #529

matt-wirtz opened this issue Feb 7, 2024 · 2 comments
Assignees
Labels
WT1 To be discussed at the next meeting
Milestone

Comments

@matt-wirtz
Copy link
Collaborator

API Version

1.5

Summary

The header of a request must include a field: API
This field should more precisely define which API definitions are used. It was intended to support different variants, e.g. country specific. This is no longer envisioned.

Possible Solution

Remove this field from the request header.

@edwinvandenbelt edwinvandenbelt added the WT1 To be discussed at the next meeting label Feb 14, 2024
@edwinvandenbelt edwinvandenbelt added this to the 2.0 milestone Feb 14, 2024
@edwinvandenbelt edwinvandenbelt self-assigned this Feb 14, 2024
@edwinvandenbelt
Copy link
Collaborator

In https://github.com/TOMP-WG/TOMP-API/tree/transmodel-v1 the fields API and API Version are dropped. The API is obvious, the version number must be (defacto standard) used in the url.

@edwinvandenbelt
Copy link
Collaborator

584f1c5

@edwinvandenbelt edwinvandenbelt moved this to In review V2.0 in WT1 Kanban Jun 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
WT1 To be discussed at the next meeting
Projects
Status: In review V2.0
Status: In review V2.0
Development

No branches or pull requests

2 participants