-
Notifications
You must be signed in to change notification settings - Fork 29
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
Alignment of Device schema #105
Comments
Should we make an artifact in the repository containing this model definition as presented here?
|
@rartych I think it will be good idea not just for Device schema, but also for other schemas that are used across multiple APIs. |
Also to note that API Design Guidelines defines syntax style for path parameters and query parameters, but not for POST (non-URL) parameters. This is especially relevant for Device Identifiers, as the Guidelines correctly state that POST should be used for sensitive data: "The classification of data tagged as sensitive should be assessed for each API project, but might include the following examples:
So the decision in this issue should trigger a new sub-section in the Guidelines (after section 3.4) to define syntax style for POST parameters. |
Related to my comment above: a similar guidance is needed Device Identifiers are passed as HTTP Headers. The POST or GET for transferring sensitive or complex data section of the API guidelines states: "When using GET, transfer the data using headers, which are not routinely logged or cached" Headers are typically Uppercase/hyphen-separated, and are documented that way in the HTTP Headers Definition section of the guidelines. Hence Device Identifier Headers would follow this convention:
|
@Kevsy Could you open a new issue for that? |
Done, #120 |
fixed by PR#107 |
Problem description
The Device schema originally modeled in QoD WG, to identify the device, has been replicated in many other APIs. In the next release v0.10, there are some enhancements and corrections that are also applicable to the rest of APIs. Specifically, IP addresses have been remodeled so now single IP addresses are expected. Descriptions have been updated accordingly, and patterns have been removed.
Details: camaraproject/QualityOnDemand#230
Possible evolution
Review and align the model in other APIs that make use of it. You can check the latest version in QoD v0.10:
https://github.com/camaraproject/QualityOnDemand/blob/main/code/API_definitions/qod-api.yaml#L895 (line may change)
For DeviceIPv6Adddres, the sentence in description "The session shall apply to all IP flows between the device subnet and the specified application server, unless further restricted by the optional parameters devicePorts or applicationServerPorts." only applies to QoD, so it has to be removed for other APIs.
The text was updated successfully, but these errors were encountered: