fix: remove inspecting text/html in sourcecraft inspector
Metadata
Current evaluation
Merged to resolve issue #228 by removing text/html inspection in the sourcecraft inspector, preventing accidental processing of non-git requests. Approved by two reviewers and passed CI checks.
Suggested action: —
No scores available.
Issue body
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
It fixes the issue, when `sourcecraft` inspector runs accidentally on non-git requests
Closes: #228
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged to resolve issue #228 by removing text/html inspection in the sourcecraft inspector, preventing accidental processing of non-git requests. Approved by two reviewers and passed CI checks. |
Update history
| Date | Change |
|---|---|
| created |
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #222 test: enable sourcecraft inspector tests | fetch-service | merged | Merged changes enabling sourcecraft inspector tests disabled during refactoring. Approved by two reviewers, passed CI, and updated four files. Deb inspector tests require separate handling. | |
| #219 docs: document sourcecraft inspector | fetch-service | merged | Merged documentation for the sourcecraft inspector. Approved by two reviewers, the changes were successfully integrated, resolving CRAFT-3337. | |
| #217 feat: add sourcecraft inspector | fetch-service | merged | Merged after two approvals. Introduces the sourcecraft inspector and resolves repository cloning issues with the fetch-service. Closes #211 and was integrated following #216. | |
| #5795 fix: properly ignore source-code field in metadata linter | snapcraft | merged | Merged to resolve issue #5794 by properly ignoring the source-code field in the metadata linter. Approved by two reviewers, passed all CI checks, and verified working. | |
| #245 feat(inspectors): make sourcecraft url origin configurable | fetch-service | merged | Merged changes making the Sourcecraft URL origin configurable in the inspectors module, resolving issue #218. Approved by two reviewers and modified nine files. |