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

feat(minio): add client info and user header to artifact binary fetcher #978

Conversation

jvallesm
Copy link
Collaborator

Because

  • The artifact binary fetcher interacted with MinIO without providing the app and user information that we need for the audit logs.

This commit

  • Uses the latest version of x/minio to provide such information.

@jvallesm jvallesm self-assigned this Feb 19, 2025
Copy link

linear bot commented Feb 19, 2025

@jvallesm jvallesm force-pushed the jvalles/ins-7319-log-requirements-for-ephi-data-access-and-modifications branch from bed509e to ba7ab95 Compare February 19, 2025 15:39
@jvallesm jvallesm force-pushed the jvalles/ins-7319-log-requirements-for-ephi-data-access-and-modifications branch from ba7ab95 to c3f6757 Compare February 20, 2025 06:10
@jvallesm jvallesm force-pushed the jvalles/ins-7319-log-requirements-for-ephi-data-access-and-modifications branch from c3f6757 to bda1bc3 Compare February 20, 2025 06:44
@jvallesm jvallesm marked this pull request as ready for review February 20, 2025 06:44
@jvallesm jvallesm requested a review from donch1989 as a code owner February 20, 2025 06:44
@jvallesm jvallesm merged commit 78c9c1f into main Feb 20, 2025
11 checks passed
@jvallesm jvallesm deleted the jvalles/ins-7319-log-requirements-for-ephi-data-access-and-modifications branch February 20, 2025 07:19
jvallesm pushed a commit that referenced this pull request Feb 25, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.51.0-beta](v0.50.0-beta...v0.51.0-beta)
(2025-02-25)


### Features

* **all:** rename VDP to pipeline
([#963](#963))
([8ba570a](8ba570a))
* **component:** support metadata filter in artifact component
([#979](#979))
([624029a](624029a))
* **Docling:** prefetch model artifacts
([#964](#964))
([c9ff323](c9ff323))
* **document:** convert PDF to Markdown with Docling
([#959](#959))
([a9dbf55](a9dbf55))
* **document:** log execution times for benchmarking
([#969](#969))
([ac3e2c3](ac3e2c3))
* **init:** remove preset pipeline downloader
([#970](#970))
([11f8f5c](11f8f5c))
* **minio:** add client info and user header to artifact binary fetcher
([#978](#978))
([78c9c1f](78c9c1f))
* **minio:** add service name and version to MinIO requests
([#976](#976))
([39c66cd](39c66cd))
* **minio:** log MinIO actions with requester
([#972](#972))
([8ba353e](8ba353e))
* **perplexity:** add new Sonar models
([#957](#957))
([2699679](2699679))
* **recipe:** rename `format` to `type` in variable section
([#971](#971))
([88ead91](88ead91))
* **x:** update MinIO package to delegate audit logs
([#973](#973))
([f81287b](f81287b))


### Bug Fixes

* **ci:** registry image build
([#960](#960))
([3a56698](3a56698))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants