Skip to content

Issue 49 cors - #62

Merged
ZhuchkaTriplesix merged 3 commits into
devfrom
issue-49-cors
Apr 26, 2026
Merged

ZhuchkaTriplesix merged 3 commits into
devfrom
issue-49-cors

Conversation

@ZhuchkaTriplesix

@ZhuchkaTriplesix ZhuchkaTriplesix commented Apr 26, 2026 •

Copy link
Copy Markdown
Member

Summary

  • What this PR does (1–3 sentences).
  • If it closes a ticket: Closes #N (replace N).

Base branch

  • This PR targets dev, not main (unless maintainers asked for a hotfix).

Checklist

  • cargo build (and cargo clippy if you touched Rust)
  • maturin develop + tests as in docs/development.md (e.g. pytest from a clean cwd / installed wheel)
  • No unrelated drive-by refactors; commits are atomic and scoped

Note on .github/ISSUE_BACKLOG/

If you only touch issue template files under .github/ISSUE_BACKLOG/ (not application code), say so in the summary. Prefer a separate PR for backlog-only edits when possible.

- Add oxyroute.cors with CORSConfig, preflightResponse, apply_cors(chain=)
- App.set_cors and exports; Rust merges response_header_pairs after handlers/middleware
- Fix Py<PyAny> clone under GIL, PyO3 0.25 extract + merge_cors_to_handler_map
- tests/test_cors.py, docs/cors.md, backlog and feature table updates
- Ensure .venv with PY_BOOTSTRAP; UV_PYTHON points at .venv to avoid PEP 668 on /usr
- One shot: sync, ruff, rustfmt+clippy, maturin develop --uv, pytest in temp dir
- AGENTS.md and git-workflow: prefer make test, human runs before push
- oxyroute: RUF022 __all__ order, ruff format cors.py
@ZhuchkaTriplesix ZhuchkaTriplesix linked an issue Apr 26, 2026 that may be closed by this pull request

@ZhuchkaTriplesix ZhuchkaTriplesix left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

123

@ZhuchkaTriplesix ZhuchkaTriplesix added this to the v0.2.0 milestone Apr 26, 2026
@ZhuchkaTriplesix ZhuchkaTriplesix self-assigned this Apr 26, 2026
@ZhuchkaTriplesix
ZhuchkaTriplesix merged commit bdc5647 into dev Apr 26, 2026
16 checks passed
@ZhuchkaTriplesix
ZhuchkaTriplesix deleted the issue-49-cors branch April 27, 2026 09:17
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.

feat: first-class CORS helper (middleware / preflight)

1 participant