Skip to content

feat: allows cross origin get requests from github dashboard#93

Merged
semmet95 merged 1 commit into
mainfrom
feature/allow-cors
Apr 26, 2026
Merged

feat: allows cross origin get requests from github dashboard#93
semmet95 merged 1 commit into
mainfrom
feature/allow-cors

Conversation

@semmet95
Copy link
Copy Markdown
Contributor

@semmet95 semmet95 commented Apr 26, 2026

Summary by cubic

Enable CORS so the GitHub dashboard at https://satyalens.github.io can call the API. Allows read-only access from the dashboard without proxying.

  • New Features

    • Add github.com/gin-contrib/cors middleware.
    • Allow origin: https://satyalens.github.io.
    • Allow methods: GET, OPTIONS.
    • Allow headers: Content-Type, X-API-Key.
    • Allow credentials: true.
  • Dependencies

    • Upgrade github.com/gin-gonic/gin to v1.12.0.
    • Add github.com/gin-contrib/cors v1.7.7.
    • Update several indirect deps to latest compatible versions.

Written for commit bcd8696. Summary will update on new commits.

Signed-off-by: Amit Singh <singhamitch@outlook.com>
@semmet95 semmet95 marked this pull request as ready for review April 26, 2026 16:49
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 3 files

Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name="cmd/app/main.go">

<violation number="1" location="cmd/app/main.go:81">
P2: Avoid hardcoding the allowed CORS origin; load it from configuration/env so origin policy can be changed per environment without code edits.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

Comment thread cmd/app/main.go
@semmet95 semmet95 merged commit de254a7 into main Apr 26, 2026
5 checks passed
@semmet95 semmet95 deleted the feature/allow-cors branch April 26, 2026 16:55
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