-
Notifications
You must be signed in to change notification settings - Fork 110
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
solr-operator does not support authentication as far back as Solr 6.6.6 #3524
Comments
@adborden and I worked on this today. We confirmed the problem in the dev environment: Solr 6.6.6 /admin does not require credentials. (The test to see if auth credentials work doesn't check to see if supplying bad/no credentials fails.) |
We investigated more and eventually filed an upstream bug with the solr-operator documenting our findings. |
We pushed the improved (and now failing) test up on the |
The latest version of the brokerpak added the capability to logout of a session in the admin UI in order to address a compliance finding... but that change only applies for newer versions of Solr than we can currently use for our CKAN apps, so it's more of an academic than practical point. We're not using the admin UI for data.gov, and the client apps will always supply credentials with each request to the API in any case. I think while the upstream bug is pending, we can just downgrade the deployed brokerpak to the previous version that used nginx-ingress to add authentication. That would unblock using brokered Solr instances with our apps. |
Can anyone speak to his questions about how CKAN uses Solr? Does it use Solr in really simple ways, or is it explicitly just not compatible with newer versions? |
Workaround for the issue described here: GSA/data.gov#3524 (comment)
@adborden said in Slack:
|
Workaround for the issue described here: GSA/data.gov#3524 (comment)
It looks like someone got it working, a few notes are made... |
Oh ho ho ho, if only it were that easy... The previous solr brokerpak expects solr-operator 0.3.0, but the eks brokerpak installs 0.4.0, which means the previous solr brokerpak doesn't work. ARGH. |
That seems like pretty minor changes. Any guesses why upstream CKAN wouldn't have made the move to 7.x if that's all it takes? |
This might be an easier lift for solr v7: ckan/ckan#4387 |
I posted on CKAN gitter about SOLR 8 on where we can track CKAN discussions, or if these conversations died if we can get caught up. |
CKAN PR to move to SOLR 8 (looks minimal, but still in draft mode): ckan/ckan#6530 |
Just noting here that the upstream PR seems to be getting close attention and is proceeding rapidly. 🤞 |
Environment: cloud.gov
Solr Version: 6.6.6
During our release of Solr on cloud.gov, when an instance of Solr was provisioned and the url was visited, the Solr Admin UI was not restricted by authentication features. Upon inspection, the authentication plugin was not configured.
Based on the results from the implementation of authentication,
How to reproduce
Expected behavior
Login page hit. Credentials accepted. Login successful.
Actual behavior
No login page. Open access.
Sketch
[Notes or a checklist reflecting our understanding of the selected approach]
Important resources:
The text was updated successfully, but these errors were encountered: