Fix Solr OKP default URL missing /solr path#1431
Fix Solr OKP default URL missing /solr path#1431are-ces wants to merge 1 commit intolightspeed-core:mainfrom
Conversation
The default Solr base URL was missing the /solr path prefix, causing 404 errors when connecting to Solr collections. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
|
Note Currently processing new changes in this PR. This may take a few minutes, please wait... ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (1)
✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip You can disable the changed files summary in the walkthrough.Disable the |
|
Caution Failed to replace (edit) comment. This is likely due to insufficient permissions or the comment being deleted. Error details |
PR Reviewer Guide 🔍Here are some key observations to aid the review process:
|
|
This PR adds |
Description
The default Solr base URL (
RH_SERVER_OKP_DEFAULT_URL) was missing the/solrpath prefix, causing404 Not Founderrors when the OKP provider connected to Solr collections (e.g.http://localhost:8081/portal-rag/selectinstead ofhttp://localhost:8081/solr/portal-rag/select).Type of change
Tools used to create PR
Related Tickets & Documents
Checklist before requesting a review
Testing
Tested using solr as knowledge base
Summary by CodeRabbit
/solrpath segment to the base URL. This ensures the correct endpoint is used when no custom configuration is provided.