Skip to content

feat(ci): enable dependabot version updates#3

Open
rcsheets wants to merge 32 commits intomainfrom
feat/ci/dependabot-version-updates
Open

feat(ci): enable dependabot version updates#3
rcsheets wants to merge 32 commits intomainfrom
feat/ci/dependabot-version-updates

Conversation

@rcsheets
Copy link
Copy Markdown
Contributor

No description provided.

@rcsheets rcsheets requested a review from grue March 13, 2025 03:05
@rcsheets rcsheets self-assigned this Mar 13, 2025
@rcsheets rcsheets requested a review from RadicalMagenta March 13, 2025 03:06
@rcsheets
Copy link
Copy Markdown
Contributor Author

Code scanning results are dependent on #1

grue
grue previously approved these changes Mar 15, 2025
Copy link
Copy Markdown

@grue grue left a comment

Choose a reason for hiding this comment

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

straightforward, looks good

rcsheets added 24 commits March 16, 2025 07:47
- Add ObjectStore interface for better testability

- Create robust cleanRequestPath function

- Add comprehensive test suite for path handling

- Improve directory traversal prevention
- Clean up unnecessary mock bucket struct in test file
- Simplify test mocking infrastructure
- Replace string-based logging with structured map payload
- Add more context to log entries including operation type and path
- Improve error logging granularity in GCS server HTTP handler
- Added github.com/davecgh/go-spew v1.1.1, github.com/pmezard/go-difflib v1.0.0, and github.com/stretchr/testify v1.10.0 to go.mod
- Updated go.sum to reflect the new versions of added dependencies
@rcsheets
Copy link
Copy Markdown
Contributor Author

straightforward, looks good

Your approval expired because of the repo rules. Would you mind approving again?

🙇‍♂️

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