Skip to content

Sanitize invalid repository owner website URLs#1598

Open
AgentGoose32 wants to merge 1 commit intoecosyste-ms:mainfrom
AgentGoose32:polar-1061-owner-url-cleanup
Open

Sanitize invalid repository owner website URLs#1598
AgentGoose32 wants to merge 1 commit intoecosyste-ms:mainfrom
AgentGoose32:polar-1061-owner-url-cleanup

Conversation

@AgentGoose32
Copy link
Copy Markdown

Refs #1061

Summary

  • Sanitizes repository owner website values before caching repos.ecosyste.ms owner metadata on packages.
  • Drops non-HTTP/HTTPS or unparseable owner website strings instead of returning plain descriptions through the package API.
  • Adds model coverage for invalid and valid owner website values.

Validation

  • ruby -c app/models/package.rb
  • ruby -c test/models/package_test.rb
  • git diff --check

bundle exec ruby -Itest test/models/package_test.rb -n '/owner website/' is blocked locally because this checkout requires Bundler 4.0.10 from Gemfile.lock, which is not available in the system Ruby environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant