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

Add support for multiple REST API servers #25

Closed
3 tasks done
GregorDeCillia opened this issue Mar 7, 2022 · 0 comments · Fixed by #23
Closed
3 tasks done

Add support for multiple REST API servers #25

GregorDeCillia opened this issue Mar 7, 2022 · 0 comments · Fixed by #23
Assignees
Labels
Milestone

Comments

@GregorDeCillia
Copy link
Contributor

GregorDeCillia commented Mar 7, 2022

There are now multiple internal STATcube API servers running inside our firewalls. Extend the package in order to

  • manage several keys in sc_key()
  • Automatically detect the correct server based on naming conventions
  • Adapt the generation of hashes to vary depending on the server
GregorDeCillia added a commit that referenced this issue Mar 7, 2022
previsously, sc_key was always using the
environment variable STATCUBE_KEY to
return/store the key value

since we have multiple servers (see #25)
it is now possible to use

- STATCUBE_KEY_EXT for the external server
- STATCUBE_KEY_RED for the editing server
- STATCUBE_KEY_PROD for the production server

part of #25
GregorDeCillia added a commit that referenced this issue Mar 7, 2022
GregorDeCillia added a commit that referenced this issue Mar 7, 2022
and auto-detect the appropriate server in
cases where this can be inferred from the
request payload

part of #25
@GregorDeCillia GregorDeCillia changed the title Add support for multiple servers Add support for multiple REST API servers Mar 7, 2022
@GregorDeCillia GregorDeCillia added this to the Version 1.0 milestone Mar 7, 2022
@GregorDeCillia GregorDeCillia self-assigned this Mar 7, 2022
@GregorDeCillia GregorDeCillia linked a pull request Mar 7, 2022 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant