Add support for Bearer Token Authorization in Solr SQLAlchemy driver

aadel / sqlalchemy-solr

Apache Solr dialect for SQLAlchemy

MIT License

9 stars 7 forks source link

Add support for Bearer Token Authorization in Solr SQLAlchemy driver #15

Closed rohitpawar2811 closed 10 months ago

rohitpawar2811 commented 10 months ago

This commit introduces support for using bearer tokens as part of the authentication mechanism in the SQLAlchemy Solr driver. The changes include:

Added a new method to handle the creation of a session with proper authorization, allowing the use of bearer tokens.
Updated the existing authentication logic to conditionally include the bearer token in the authorization headers.
Ensured that the session can be utilized in other methods within the class. @aadel

aadel commented 10 months ago

Hi @rohitpawar2811 Thank you for your contribution! LGTM however I had to replace all url.query occurrences with kwargs in http.py to get to work on Superset 3.1.0rc2. Otherwise, when using url.query, the following error is returned:

[SQL: connect() got an unexpected keyword argument 'token']

What is the environment used at your end?