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

addition of templated endpoints mechanism #28

Closed
wants to merge 1 commit into from

Conversation

Bonnarel
Copy link
Contributor

We miss a mechanism to describe variable RESTful endpoints. This can be achieved by a templating mechanism described here. It's related to issue #27

@mbtaylor
Copy link
Member

I'm not convinced this is a good idea in any case (is the additional complication justified by the benefits?), but if it is to be accepted, it needs to be much more carefully worded than

"The templating scheme adopted in the value attribute of this PARAM is following the appropriate IETF RFC \citep{std:RFC6570}."

RFC6570 is fairly complicated, and IMHO it is not a good idea to require DataLink parsing software to incorporate a full RFC6570 template parser. At the very least it's necessary to say which RFC6570 Levels are permitted, and to be explicit about how table content is turned into template variables. An example would also be a good idea.

Also, I don't understand why this PR pulls in provdm, provsap and provtap citations alongside RFC6570.

@Bonnarel
Copy link
Contributor Author

Bonnarel commented Nov 8, 2019 via email

@msdemlei
Copy link
Collaborator

msdemlei commented Nov 8, 2019 via email

@Bonnarel
Copy link
Contributor Author

Bonnarel commented Nov 12, 2019 via email

@msdemlei
Copy link
Collaborator

msdemlei commented Nov 13, 2019 via email

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.

3 participants