Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add go-chi detector & analyzer #529

Merged
merged 7 commits into from
Feb 4, 2025
Merged

Add go-chi detector & analyzer #529

merged 7 commits into from
Feb 4, 2025

Conversation

hahwul
Copy link
Member

@hahwul hahwul commented Feb 3, 2025

Added go-chi (golang web framework) detector and analyzer

Signed-off-by: HAHWUL [email protected]

@hahwul hahwul self-assigned this Feb 3, 2025
@github-actions github-actions bot added 🔎 detector Issue for Detector 💊 spec Issue for test codes labels Feb 3, 2025
@hahwul hahwul linked an issue Feb 3, 2025 that may be closed by this pull request
@hahwul hahwul added this to the v0.20.0 milestone Feb 3, 2025
@github-actions github-actions bot added the 🔬 analyzer Issue for Analyzer label Feb 3, 2025
@hahwul
Copy link
Member Author

hahwul commented Feb 3, 2025

Test

Screenshot 2025-02-03 at 11 22 34 PM

@hahwul hahwul requested a review from ksg97031 February 3, 2025 14:24
@hahwul hahwul marked this pull request as ready for review February 3, 2025 14:24
@hahwul hahwul marked this pull request as draft February 3, 2025 14:27
@hahwul
Copy link
Member Author

hahwul commented Feb 3, 2025

@ksg97031 cc @boundmania
While coding, I found that handling the headers was getting a bit tricky, so I decided to extract just the API endpoint. I'd love to hear your thoughts!

    go_chi
      ➔ framework: Chi
      ➔ language: Go
      ➔ similar: ["chi", "go-chi", "go_chi"]
      ➔ supported:
        └── endpoint: true
        └── method: true
        └── params: {:query => true, :path => true, :body => true, :header => false, :cookie => false}
        └── static_path: false
        └── websocket: false

@hahwul hahwul marked this pull request as ready for review February 3, 2025 14:56
Signed-off-by: HAHWUL <[email protected]>
Copy link
Member

@ksg97031 ksg97031 left a comment

Choose a reason for hiding this comment

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

LGTM!

@ksg97031 ksg97031 merged commit 51eaf11 into main Feb 4, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔬 analyzer Issue for Analyzer 🔎 detector Issue for Detector 💊 spec Issue for test codes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add go-chi/chi detector and analyzer
2 participants