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

Remove includeAuthorization parameter for GET services #182

Open
KevLoh opened this issue Nov 27, 2018 · 3 comments
Open

Remove includeAuthorization parameter for GET services #182

KevLoh opened this issue Nov 27, 2018 · 3 comments

Comments

@KevLoh
Copy link

KevLoh commented Nov 27, 2018

As agreed in BCF Hackathon 2018 p2, we should remove the additional includeAuthorization parameter in GET services (https://github.com/BuildingSMART/BCF-API#428-determining-allowed-topic-modifications).

Always return authorization, topic_status etc..

@jasollien jasollien added this to the V3.0 milestone Nov 27, 2018
@ykulbak ykulbak modified the milestones: V3.0, Future Release Nov 25, 2020
@GeorgDangl GeorgDangl self-assigned this Jun 5, 2023
@GeorgDangl
Copy link
Member

@ykulbak @jasollien could you guys give a bit of feedback if it would be easy to do from the server side? Easy meaning, it's not going to slow down all requests when we always include the authorization information.

@GeorgDangl
Copy link
Member

We got feedback from Oracle and Catenda that this should not be a problem, meaning we can always include the authorization parameters in responses and simplify the API.

@jasollien
Copy link
Contributor

Agreed in todays telecon to do this change in the hackathon

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants