google-research / CVD-paper-mobile-camera-example

Apache License 2.0
6 stars 4 forks source link

Provide Bearer token Authentication mechanism to call Blob store APIs #45

Closed chandrashekar-s closed 6 months ago

chandrashekar-s commented 7 months ago

Provide token based authentication mechanism for invoking cloud storage APIs.

chandrashekar-s commented 7 months ago

Refer to this doc for the implementation of the API.

khyativyasargus commented 6 months ago

@MJ1998 Need to expose Interface to set token for MinIO client to support token based authentication with external IDP. Also can we have option to choose either token is bearer or not?

MJ1998 commented 6 months ago

@khyativyasargus

Above is first version of the PR. How we are imagining this is to provide CredentialsProviderAuthenticator & CustomIDPAuthenticator interfaces which can be implemented by application layer for OpenIDP and custom IDP respectively.

In your case you would need to implement the latter and provide the sdk with refreshed token, RoleArn and duration in seconds.

The former one is for IDPs that are supported by minio, for example, OpenIDP or LDAP IDP where one can directly provide already available Provider implementations, for example, io.minio.credentials.LdapIdentityProvider

MJ1998 commented 6 months ago

This has been fixed by PR #54 . Closing this.

MJ1998 commented 6 months ago

Pr is yet to be merged.