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

取り込み #44

Merged
merged 13 commits into from
Sep 27, 2024
Merged

取り込み #44

merged 13 commits into from
Sep 27, 2024

Conversation

kynmh69
Copy link
Owner

@kynmh69 kynmh69 commented Sep 27, 2024

プルリクエストのタイトル

概要

  • このプルリクエストの目的と主な変更点を簡潔に説明してください。

issueの番号

変更内容

  • 具体的な変更点をリスト形式で明記してください。

Go言語に関するチェックリスト

  • コードは gofmt でフォーマットされている
  • go vet を実行して潜在的な問題がないことを確認
  • ユニットテストが追加/更新されている
  • パフォーマンスに影響する変更はベンチマークを通して評価されている
  • 依存関係の更新は適切に管理されている
  • エラーハンドリングが適切に行われている
  • Goのコーディング規約に準拠している
  • バグが修正されていること

影響範囲とリスク

  • この変更が及ぼす影響と潜在的なリスクについて説明してください。

テスト戦略

  • どのようにしてこれらの変更が正しいことをテストしたか説明してください。

追加情報 (任意)

  • レビュワーに伝えたいその他の情報や、関連するIssueなどがあればこちらに記載してください。

kynmh69 and others added 13 commits September 27, 2024 23:13
#40
Disable ECR deployment step in GitHub Actions workflow.

Commented out the ECR deployment steps including AWS credentials setup, Docker Buildx setup, and Docker image build and push actions. This change pauses the deployment to ECR, allowing for adjustments or testing without triggering deployments.
IntelliJ IDEA project files should not be tracked in Git. This update helps keep the repository clean and prevents accidental commits of local IDE configurations.
Commented out the deploy steps in the `docker-publish-key-manager.yml` workflow. This change prevents the deployment to Amazon ECR, including steps for build setup, AWS credentials configuration, and Docker image building and pushing.
Commented out the deploy step in the GitHub Actions workflow file. This includes disabling steps related to Docker build, AWS credentials configuration, and ECR login. This update halts automatic deployment to ECR in the current setup.
fix: update Dockerfile for Go build and deployment

- Use golang:1.23 for building the updater
- Copy source files to /go/app/
- Set working directory to /go/app and download dependencies
- Build the updater binary with CGO_ENABLED=0
- Use alpine:latest for the final image
- Add zoneinfo.tar.gz to the root directory
- Copy the built binary to /usr/local/bin/
- Set the default command to run the binary
Bumps the npm_and_yarn group with 1 update in the /src/key_management directory: [braces](https://github.com/micromatch/braces).


Updates `braces` from 3.0.2 to 3.0.3
- [Changelog](https://github.com/micromatch/braces/blob/master/CHANGELOG.md)
- [Commits](micromatch/braces@3.0.2...3.0.3)

---
updated-dependencies:
- dependency-name: braces
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <[email protected]>
…anagement/npm_and_yarn-5134b82be1

Bump braces from 3.0.2 to 3.0.3 in /src/key_management in the npm_and_yarn group across 1 directory
Copy link

coderabbitai bot commented Sep 27, 2024

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

Qodana for Go

44 new problems were found

Inspection name Severity Problems
Vulnerable declared dependency 🔴 Failure 1
Unhandled error 🔶 Warning 21
Imported package name as a name identifier 🔶 Warning 2
Unused exported function 🔶 Warning 2
Unused constant 🔶 Warning 1
Unused parameter 🔶 Warning 1
Usage of Snake_Case ◽️ Notice 11
Incorrect usage of 'fmt.Printf' and 'fmt.Println' functions ◽️ Notice 2
Duplicated code fragment ◽️ Notice 1
Name starts with a package name ◽️ Notice 1
Type assertion on errors fails on wrapped errors ◽️ Notice 1

☁️ View the detailed Qodana report

Detected 71 dependencies

Third-party software list

This page lists the third-party software dependencies used in project

Dependency Version Licenses
github.com/DATA-DOG/go-sqlmock v1.5.0 BSD-3-Clause
github.com/HdrHistogram/hdrhistogram-go v1.1.2 MIT
github.com/Knetic/govaluate v3.0.1-0.20171022003610-9aa49832a739+incompatible MIT
github.com/alecthomas/template v0.0.0-20190718012654-fb15b899a751 BSD-3-Clause
github.com/alecthomas/units v0.0.0-20190924025748-f65c72e2690d MIT
github.com/beorn7/perks v1.0.1 MIT
github.com/casbin/casbin/v2 v2.64.0 Apache-2.0
github.com/cespare/xxhash/v2 v2.2.0 MIT
github.com/davecgh/go-spew v1.1.1 ISC
github.com/denisenkom/go-mssqldb v0.10.0 BSD-3-Clause
github.com/doug-martin/goqu/v9 v9.19.0 MIT
github.com/go-kit/log v0.2.1 MIT
github.com/go-logfmt/logfmt v0.5.1 MIT
github.com/go-sql-driver/mysql v1.6.0 MPL-2.0
github.com/golang-jwt/jwt v3.2.2+incompatible MIT
github.com/golang-sql/civil v0.0.0-20190719163853-cb61b32ac6fe Apache-2.0
github.com/golang/protobuf v1.5.2 BSD-3-Clause
github.com/google/go-cmp v0.5.9 BSD-3-Clause
github.com/google/gofuzz v1.0.0 Apache-2.0
github.com/google/uuid v1.6.0 BSD-3-Clause
github.com/gorilla/context v1.1.1 BSD-3-Clause
github.com/gorilla/securecookie v1.1.1 BSD-3-Clause
github.com/gorilla/sessions v1.2.1 BSD-3-Clause
github.com/jpillora/backoff v1.0.0 MIT
github.com/json-iterator/go v1.1.12 MIT
github.com/julienschmidt/httprouter v1.3.0 BSD-3-Clause
github.com/kr/pretty v0.3.1 MIT
github.com/labstack/echo-contrib v0.15.0 MIT
github.com/labstack/echo/v4 v4.11.4 MIT
github.com/labstack/gommon v0.4.2 MIT
github.com/lib/pq v1.10.1 MIT
github.com/mattn/go-colorable v0.1.13 MIT
github.com/mattn/go-isatty v0.0.20 MIT
github.com/mattn/go-sqlite3 v1.14.7 MIT
github.com/matttproud/golang_protobuf_extensions v1.0.4 Apache-2.0
github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd Apache-2.0
github.com/modern-go/reflect2 v1.0.2 Apache-2.0
github.com/mwitkow/go-conntrack v0.0.0-20190716064945-2f068394615f Apache-2.0
github.com/opentracing/opentracing-go v1.2.0 Apache-2.0
github.com/openzipkin/zipkin-go v0.4.1 Apache-2.0
github.com/pkg/errors v0.9.1 BSD-2-Clause
github.com/pmezard/go-difflib v1.0.0 BSD-3-Clause
github.com/prometheus/client_golang v1.14.0 Apache-2.0
github.com/prometheus/client_model v0.3.0 Apache-2.0
github.com/prometheus/common v0.40.0 Apache-2.0
github.com/prometheus/procfs v0.9.0 Apache-2.0
github.com/stretchr/objx v0.5.2 MIT
github.com/stretchr/testify v1.8.4 MIT
github.com/uber/jaeger-client-go v2.30.0+incompatible Apache-2.0
github.com/uber/jaeger-lib v2.4.1+incompatible Apache-2.0
github.com/valyala/bytebufferpool v1.0.0 MIT
github.com/valyala/fasttemplate v1.2.2 MIT
go.uber.org/atomic v1.10.0 MIT
golang.org/x/crypto v0.21.0 BSD-3-Clause
golang.org/x/mod v0.8.0 BSD-3-Clause
golang.org/x/net v0.23.0 BSD-3-Clause
golang.org/x/oauth2 v0.3.0 BSD-3-Clause
golang.org/x/sync v0.1.0 BSD-3-Clause
golang.org/x/sys v0.18.0 BSD-3-Clause
golang.org/x/term v0.18.0 BSD-3-Clause
golang.org/x/text v0.14.0 BSD-3-Clause
golang.org/x/time v0.5.0 BSD-3-Clause
golang.org/x/tools v0.6.0 BSD-3-Clause
golang.org/x/xerrors v0.0.0-20191204190536-9bdfabe68543 BSD-3-Clause
google.golang.org/appengine v1.6.7 Apache-2.0
google.golang.org/grpc v1.53.0 Apache-2.0
google.golang.org/protobuf v1.33.0 BSD-3-Clause
gopkg.in/alecthomas/kingpin.v2 v2.2.6 MIT
gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c BSD-2-Clause
gopkg.in/yaml.v2 v2.4.0 Apache-2.0
MIT
gopkg.in/yaml.v3 v3.0.1 Apache-2.0
MIT
Contact Qodana team

Contact us at [email protected]

@kynmh69 kynmh69 merged commit fe7cd88 into 35-feature-add-key-auth Sep 27, 2024
4 checks passed
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.

2 participants