aws / aws/amazon-redshift-python-driver
Is`OktaCredentialsProvider.handle_saml_assertion` compatible with SageMaker instances?
- Lenguaje dominante
- Python
- Estrellas
- 220
- Forks
- 86
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
I'm encountering an issue when using `redshift_connector` with Okta-based authentication inside my organisation's SageMaker environment.
In `OktaCredentialsProvider.handle_saml_assertion` I receive an error [okta_credentials_provider.py#L158](https://github.com/aws/amazon-redshift-python-driver/blob/fd81061c4dfee9ed3001be039f6b1826d8366302/redshift_connector/plugin/okta_credentials_provider.py#L158) despite receiving a 200 OK and a response of length > 2k.
Upon investigating the logs it seems to be down to a browser or Javascript issue? I've tried on both SageMaker Studio Code Editor and SageMaker Notebook instances and get the same issue.
It's also strange that `OktaCredentialsProvider.okta_authentication` works perfectly and I'm able to retrieve the `okta_session_token`, but the later step fails.
The same identical code works perfectly on my local machine.
I've attached the relevant part of the response below:
```
...
Javascript is required
Javascript is disabled on your browser. Please enable Javascript and refresh this page.
Refresh
...
The page has timed out
If this page does not reload automatically, please refresh your browser.
Guía de contribución
Línea de trabajo
Comienza en redshift_connector/plugin/okta_credentials_provider.py, en handle_saml_assertion, especialmente en la línea 158, y compara el manejo de la respuesta entre una máquina local y SageMaker. Confirma si la respuesta de SageMaker puede manejarse de forma compatible y documenta una prueba reproducible o el comportamiento esperado para la corrección.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- aws, python
- Área
- authentication, databases
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Estancado
- Claridad
- Necesita aclaración
- Aptitud para principiantes
- 35/100