Skip to content

Update references to hadiscover.ravensberg.org - #442

Merged
DevSecNinja merged 1 commit into
mainfrom
devsecninja-domain-migration
Oct 18, 2026
Merged

DevSecNinja merged 1 commit into
mainfrom
devsecninja-domain-migration

Conversation

@DevSecNinja

@DevSecNinja DevSecNinja commented Oct 18, 2026 •

Copy link
Copy Markdown
Owner

The public site has migrated to https://hadiscover.ravensberg.org/, but repository links, hosted search-index defaults, and the API CORS allowlist still referenced the previous domain.

Updates those references consistently across documentation, Docker/CI configuration, release-note generation, and the robots.txt sitemap example. Replaces the production CORS origin while preserving localhost access, aligns the CORS documentation with the actual allowlist, and adds regression coverage for allowed origins, preflight requests, and unrelated origins.

GitHub Pages and the repository homepage already use the new domain, so no deployment settings are changed.

Validation

  • Backend: 90 tests passed.
  • Frontend: production build passed with the new search-index URL; generated assets contain the migrated URLs.
  • No old domain references remain in tracked files.
  • All pre-commit checks passed except the bundled ShellCheck hook, whose binary is incompatible with this Mac's CPU. Native ShellCheck 0.11.0 passed on all tracked shell scripts.

Fixes #441

Summary by CodeRabbit

  • Updates
    • The site and its search index are now associated with hadiscover.ravensberg.org.
    • The “Try it now” link and screenshot destination in the README now point to the updated domain.
    • Cross-origin requests are allowed from the updated production domain and http://localhost:8080.
  • Documentation
    • Deployment and setup guidance now reflects the updated domain.

Replace the retired production domain across documentation, build defaults, release notes, and the CORS allowlist. Preserve localhost access and cover allowed and disallowed CORS requests.

Fixes #441

Co-authored-by: Copilot App <[email protected]>
Copilot-Session: 8b03f30b-f458-45b0-b631-f2e309135b6e
Copilot AI balanced review requested due to automatic review settings October 18, 2026 10:55
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions github-actions Bot added documentation Improvements or additions to documentation frontend backend docker ci/cd tests labels Oct 18, 2026
@coderabbitai

coderabbitai Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 18fbe715-fd74-47f1-9ba7-1ad36b7eb10b
📥 Commits

Reviewing files that changed from the base of the PR and between 423b71b and 81eb40f.

📒 Files selected for processing: 11
  • .github/copilot-instructions.md
  • .github/scripts/create-github-release.sh
  • .github/workflows/docker-test.yml
  • ARCHITECTURE.md
  • README.md
  • backend/README.md
  • backend/app/main.py
  • backend/tests/test_api.py
  • docker-compose.prod.yml
  • docker-compose.yml
  • frontend/public/robots.txt

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

📝 Walkthrough

Repository references and runtime configuration now use hadiscover.ravensberg.org. The backend CORS allowlist uses the new production origin, and tests cover local, production, and disallowed origins.

Changes: Production domain update

Layer / File(s) Summary
Runtime URLs and CORS
backend/app/main.py, docker-compose.prod.yml, docker-compose.yml, .github/workflows/docker-test.yml, backend/tests/test_api.py
Search-index defaults and CI build arguments now use the new domain. The backend CORS allowlist and tests cover the production and local origins, plus a disallowed origin.
Documentation and public references
.github/copilot-instructions.md, ARCHITECTURE.md, backend/README.md, README.md, .github/scripts/create-github-release.sh, frontend/public/robots.txt
Documentation, public links, release notes, and the commented sitemap URL now use the new domain.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Suggested reviewers: copilot

Merge Risk: ⚪ Minimal · up to 81eb4

The domain update appears ready to merge after normal checks.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the main change: updating repository references to the new production domain.
Linked Issues check Passed Issue #441 is directly linked and has coding requirements. The supplied change summary covers the requested public links, screenshot text, architecture and contributor documentation, Docker Compose an…
Out of Scope Changes check Passed The reported changes are limited to the domain migration in issue #441. The CORS tests, documentation updates, configuration updates, release-note update, and robots.txt update directly support the is…
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 3 files. (8 skipped: 8 …
Full details: Linked Issues check

Explanation

Issue #441 is directly linked and has coding requirements. The supplied change summary covers the requested public links, screenshot text, architecture and contributor documentation, Docker Compose and CI search-index defaults, release-note generation, and the robots.txt sitemap example. backend/app/main.py allows only http://localhost:8080 and https://hadiscover.ravensberg.org. backend/README.md documents the same origins. backend/tests/test_api.py covers both allowed origins, GET and POST preflight requests, required headers, and an unrelated origin. The supplied validation states that no old-domain references remain in tracked files.

Full details: Out of Scope Changes check

Explanation

The reported changes are limited to the domain migration in issue #441. The CORS tests, documentation updates, configuration updates, release-note update, and robots.txt update directly support the issue objectives. The summary reports no deployment-setting changes and identifies no unrelated behavior changes.

Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 3 files. (8 skipped: 8 unsupported.)

✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The domain migration is consistent, preserves the restricted CORS allowlist, and includes focused regression coverage with no blocking issues identified.

No open issues 🎉

What changed in this PR

Updates hadiscover’s public-domain references to hadiscover.ravensberg.org, addressing #441 without changing deployment settings.

Changes:

  • Aligns documentation, Docker defaults, CI build arguments, and release notes.
  • Updates the production CORS origin while preserving localhost access.
  • Adds CORS regression tests for allowed origins, preflight requests, and unrelated origins.
File Description
README.md Updates public links and screenshot description.
frontend/​public/​robots.txt Updates the commented sitemap example.
docker-compose.yml Updates the hosted search-index default.
docker-compose.prod.yml Updates the search-index default and deployment comment.
backend/​tests/​test_api.py Adds CORS regression coverage.
backend/​README.md Aligns CORS documentation with the allowlist.
backend/​app/​main.py Replaces the production CORS origin.
ARCHITECTURE.md Updates production-domain and CORS documentation.
.github/​workflows/​docker-test.yml Updates frontend build arguments.
.github/​scripts/​create-github-release.sh Updates generated release-note text.
.github/​copilot-instructions.md Updates contributor examples and domain references.

Review effort: Balanced

@DevSecNinja
DevSecNinja merged commit 6a6629e into main Oct 18, 2026
27 checks passed
@DevSecNinja
DevSecNinja deleted the devsecninja-domain-migration branch October 18, 2026 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend ci/cd docker documentation Improvements or additions to documentation frontend tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update references for the migration to hadiscover.ravensberg.org

2 participants