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

Update/Extend general documentation on Authentication #7

Open
m-mohr opened this issue Aug 25, 2021 · 5 comments
Open

Update/Extend general documentation on Authentication #7

m-mohr opened this issue Aug 25, 2021 · 5 comments
Assignees
Labels

Comments

@m-mohr
Copy link
Collaborator

m-mohr commented Aug 25, 2021

also include some screenshots for a step-by-step guide...

Would say this is a task for EGI, but not sure who I could assign this to.

@przell
Copy link
Contributor

przell commented Oct 11, 2021

currently this documentaton is pointing to terrascope: https://docs.openeo.cloud/authentication/#terrascope-registration
Is that correct/up to date?

@m-mohr
Copy link
Collaborator Author

m-mohr commented Oct 13, 2021

Yes, it seems that this is still required for now. See #21

@m-mohr
Copy link
Collaborator Author

m-mohr commented Oct 14, 2021

@jdries Clarification is required on when Terrascope registration is required (for SH collections? only for batch jobs? see the unresolved comments in #21 for details).

And once not required anymore, we need to remove the Terrascope registration completely.

@m-mohr m-mohr added the auth label Oct 14, 2021
@jdries
Copy link
Contributor

jdries commented Oct 14, 2021

I replied in the pull request, everything will also 'work' when the registration is not done:

Correct, even batch jobs will work, but might be slower in case multiple unregistered users are using it.
So it is not a huge problem if users skip this, but if they complain later on, I would need some paragraph to point to. For usage monitoring, this is also better, but that doesn't affect user experience yet.

@m-mohr
Copy link
Collaborator Author

m-mohr commented Oct 14, 2021

@jdries I've updated the authentication page to reflect what has been said in the PR #21 (Terrascope registration is optional, but may affect prior and performance), see 335fe02. Feel free to change or improve.

I'm leaving it up to VITO to move this to a separate page as proposed by @soxofaan in PR #21.

m-mohr pushed a commit that referenced this issue Oct 21, 2021
)

* Issue #7: move Terrascope registration from authentication to federation docs

* Issue #7/#24 add link from authentication to federation#registration
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants