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

IPT syncronization denied with new OIDC functionality #162

Open
vjrj opened this issue Sep 28, 2022 · 4 comments
Open

IPT syncronization denied with new OIDC functionality #162

vjrj opened this issue Sep 28, 2022 · 4 comments
Assignees

Comments

@vjrj
Copy link
Contributor

vjrj commented Sep 28, 2022

Using 3.1.0 we cannot sync to a IPT data provider, that is, using the web admin interface.

image

We can see in the logs:

[2022-09-28 08:48:31] [info] 2022-09-28 08:48:31.534 INFO --- [.0-8080-exec-27] a.o.a.ws.security.service.ApiKeyService : Rejected - no key present
[2022-09-28 08:48:31] [info] 2022-09-28 08:48:31.535 WARN --- [.0-8080-exec-27] a.o.a.c.CollectoryWebServicesInterceptor : Denying access to contacts from remote addr: 127.0.0.1, remote host: 127.0.0.1

and the api_key is configured in the collectory properties.

This was working previously to integrate the OIDC functionality.

cc @sughics .

@vjrj
Copy link
Contributor Author

vjrj commented Sep 29, 2022

Using /openapi/ and Authorization header redirects to the CAS auth login:

image

but works adding apiKey as param instead of the header.

djtfmartin added a commit that referenced this issue Aug 7, 2023
@adam-collins
Copy link
Contributor

Closing, I assume it is fixed now.

@vjrj vjrj reopened this Apr 15, 2024
@vjrj
Copy link
Contributor Author

vjrj commented Apr 15, 2024

I reopen this as the fix is in a branch not committed in develop.

@adam-collins
Copy link
Contributor

I have merged with develop, tested as well as I understand the issue, and have prepared a pull request for review.

@adam-collins adam-collins self-assigned this Apr 16, 2024
vjrj pushed a commit that referenced this issue Apr 18, 2024
vjrj pushed a commit to living-atlases/collectory that referenced this issue Oct 29, 2024
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

No branches or pull requests

2 participants