Denis Kirillov dkirillov
0 followers · 0 following
  • Joined on 2023-03-06
Repositories
18
 Projects Packages Code Public activity Starred repositories
1
dkirillov commented on pull request TrueCloudLab/frostfs-http-gw#153 2024-11-19 12:51:56 +00:00
[#151] Index page for FrostFS containers

Then if we create object 5rnoerMhK3kSxDEUh2FrSQvSW86P2deahSifFLxFBF9v into bucket my-bucket and I wanted to get it by http-gw like `curl http:///get/my-bucket/5rnoerMhK3kSxDEUh2FrSQvSW…

dkirillov pushed to featue/553-dont_use_basic_acl at dkirillov/frostfs-s3-gw 2024-11-19 12:46:10 +00:00
b1775f9478 [#553] authmate: Add retryer to create access box
4fa45bdac2 [#553] authmate: Don't use basic acl
Compare 2 commits »
dkirillov pushed to featue/553-dont_use_basic_acl at dkirillov/frostfs-s3-gw 2024-11-19 12:12:11 +00:00
dce34ddb9b [#553] authmate: Add retryer to create access box
7c2f247442 [#553] authmate: Don't use basic acl
Compare 2 commits »
dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#554 2024-11-19 12:11:31 +00:00
[#553] authmate: Don't use basic acl

Yes, we have to add explicit rule for container owner. We will create AccessBox there and we need access.

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#554 2024-11-19 12:06:47 +00:00
[#553] authmate: Don't use basic acl

Oh, I just extended your branch. I'll fix style

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#554 2024-11-19 12:06:08 +00:00
[#553] authmate: Don't use basic acl

Sure

1d57092a4b [#556] Check bucket name not only during creation
dkirillov created pull request TrueCloudLab/frostfs-s3-gw#556 2024-11-19 11:49:34 +00:00
[#555] Check bucket name not only during creation
543d47db4b [#XX] Check bucket name not only during creation
dkirillov pushed to featue/553-dont_use_basic_acl at dkirillov/frostfs-s3-gw 2024-11-19 08:22:47 +00:00
ed4cc417b7 [#553] authmate: Add retryer to create access box
dkirillov approved TrueCloudLab/frostfs-node#1504 2024-11-19 07:49:44 +00:00
Make container service set correct APE-request targets before the check
dkirillov opened issue TrueCloudLab/frostfs-node#1503 2024-11-18 14:55:31 +00:00
Should AddChain update/invalidate APE cache?
dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#551 2024-11-18 14:09:03 +00:00
[#543] Add md5 sse-c S3Tests compatability

What about adding tests?

dkirillov commented on pull request TrueCloudLab/frostfs-http-gw#153 2024-11-18 13:53:17 +00:00
[#151] Index page for FrostFS containers

By the way, why do we ever need to set success status?

dkirillov commented on pull request TrueCloudLab/frostfs-http-gw#153 2024-11-18 13:50:53 +00:00
[#151] Index page for FrostFS containers

And why we set status here rather than in h.browseObjects?

dkirillov commented on pull request TrueCloudLab/frostfs-http-gw#153 2024-11-18 13:49:40 +00:00
[#151] Index page for FrostFS containers

Why do we need this if we have the same on line 433 ?

dkirillov commented on pull request TrueCloudLab/frostfs-http-gw#153 2024-11-18 13:41:51 +00:00
[#151] Index page for FrostFS containers

Why don't we check cid also?

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#541 2024-11-18 13:26:08 +00:00
[#541] Use default value if config param is unset after SIGHUP

Please, rebase

dkirillov approved TrueCloudLab/frostfs-http-gw#164 2024-11-18 12:39:07 +00:00
Add tracing attributes

LGTM