Skip to content

Conversation

@cthulhu-rider
Copy link
Contributor

@cthulhu-rider cthulhu-rider commented Jun 19, 2025

tested S3 and REST GWs require signatures to be presented. Passed tests assert proposed behavior is backward compatible. Note that clients which do not set API version are also served with signatures

@cthulhu-rider cthulhu-rider force-pushed the unsigned-get branch 2 times, most recently from 9dbdea1 to 6369291 Compare June 19, 2025 12:12
@codecov
Copy link

codecov bot commented Jun 19, 2025

Codecov Report

Attention: Patch coverage is 0% with 89 lines in your changes missing coverage. Please review.

Project coverage is 20.97%. Comparing base (9b9535a) to head (ec7c76e).
Report is 7 commits behind head on master.

Files with missing lines Patch % Lines
pkg/services/object/server.go 0.00% 89 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3406   +/-   ##
=======================================
  Coverage   20.97%   20.97%           
=======================================
  Files         707      707           
  Lines       52843    52875   +32     
=======================================
+ Hits        11083    11090    +7     
- Misses      40962    40989   +27     
+ Partials      798      796    -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@cthulhu-rider cthulhu-rider force-pushed the unsigned-get branch 7 times, most recently from 60620b4 to 7b4e082 Compare June 23, 2025 14:51
@cthulhu-rider cthulhu-rider marked this pull request as ready for review June 23, 2025 14:53
@cthulhu-rider cthulhu-rider force-pushed the unsigned-get branch 2 times, most recently from 13f8508 to f1ea991 Compare June 25, 2025 08:05
New version makes SN, IR and CLI to:
 - ignore signatures
 - verify object checksums
from object GET/HEAD responses via API client from SDK.

Refs #3396.

Signed-off-by: Leonard Lyubich <[email protected]>
Continues 041cdce. Follow
nspcc-dev/neofs-sdk-go#719 in request proxy case.

Refs #3396.

Signed-off-by: Leonard Lyubich <[email protected]>
Continue 041cdce for request proxy case.

Refs #3396.

Signed-off-by: Leonard Lyubich <[email protected]>
Continue debfe6e for object payload.

Refs #3396.

Signed-off-by: Leonard Lyubich <[email protected]>
Same as f2a770a.

Signed-off-by: Leonard Lyubich <[email protected]>
@roman-khimov roman-khimov merged commit bc71582 into master Jun 25, 2025
19 of 21 checks passed
@roman-khimov roman-khimov deleted the unsigned-get branch June 25, 2025 08:31
@roman-khimov roman-khimov mentioned this pull request Jul 2, 2025
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.

3 participants