← Back to issue list

fix: remove inspecting text/html in sourcecraft inspector

View original Github issue

Metadata

Project
fetch-service
Number
#229
Type
pull request
State
merged
Author
dariuszd21
Labels
Created
Updated
Closed

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.
76%
#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.
75%
#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.
73%
#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.
72%
#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.
71%