fix(register): repoint 32 dead configuration URLs to GitHub - #748
Merged
Conversation
Every `@self.configuration` in this register pointed at https://codeberg.org/Conduction/opencatalogi/raw/branch/master/ apps-extra/opencatalogi/lib/Settings/publication_register_magic.json which returns **HTTP 404**. Verified with curl, not assumed from the host name. Three separate things had rotted at once: host ...... codeberg is no longer where opencatalogi lives path ...... the old monorepo layout `apps-extra/opencatalogi/...` is gone; the app is at its repo root now filename .. `publication_register_magic.json` was renamed to `publication_register.json` So repointing the host alone would still have 404'd. The replacement is the live file, confirmed HTTP 200 on main, master and development; `main` is the repo's default branch and is what this now uses. This matters because a `configuration` URL is FETCHED at register-import time. A 404 there does not announce itself as a broken link — the import simply does not get the configuration it asked for, which is the quiet-failure shape this codebase has been bitten by before. 32 occurrences, all byte-identical, replaced in one pass. JSON re-parsed clean afterwards.
Contributor
Quality Report — ConductionNL/stackiq @
|
| Check | PHP | Vue | Security | License | Tests |
|---|---|---|---|---|---|
| lint | ✅ | ||||
| phpcs | ✅ | ||||
| phpmd | ✅ | ||||
| psalm | ✅ | ||||
| phpstan | ✅ | ||||
| phpmetrics | ✅ | ||||
| eslint | ✅ | ||||
| stylelint | ✅ | ||||
| build | ✅ | ||||
| check-manifest | ✅ | ||||
| check-vue-demi | ✅ | ||||
| test-l10n | ✅ | ||||
| format | ✅ | ||||
| check-schema-l10n | ✅ | ||||
| composer | ✅ | ✅ 130/130 | |||
| npm | ✅ | ✅ 720/720 | |||
| app:check-code | ⏭️ | ||||
| info.xml | ✅ | ||||
| REUSE | ❌ | ||||
| PHPUnit | ✅ | ||||
| Newman | ⏭️ | ||||
| Playwright | ✅ | ||||
| Hydra gates | ✅ |
Quality workflow — 2026-08-26 10:06 UTC
Download the full PDF report from the workflow artifacts.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Every
@self.configurationinsoftwarecatalogus_register.json— 32 of them — pointed at a URL that returns HTTP 404.Verified with
curl, not inferred from the host name.Three things had rotted at once
apps-extra/opencatalogi/…is gone; the app sits at its repo root nowpublication_register_magic.jsonwas renamed topublication_register.jsonRepointing the host alone would still have 404'd — which is why I checked the target existed rather than doing a host swap.
Why this is not cosmetic
A
configurationURL is fetched at register-import time. A 404 there does not announce itself as a broken link: the import simply does not receive the configuration it asked for. That is the quiet-failure shape this codebase has been bitten by repeatedly — the request succeeds in the sense that something came back, and nothing reports that it was wrong.Verification
main,masteranddevelopment;mainis the repo's default branch and is what this uses.Found during a fleet sweep for shipped Codeberg references. Three other hits were checked and deliberately left alone: openbuild's forge-type dropdown (
codebergis a valid option there), opencatalogi's schema description naming forge hosts as examples, and hermiq'sMIRROR_HOSTSlist. Those are correct as written.