Denis Kirillov dkirillov
  • Joined on 2023-03-06
e7db3c80e1 [#73] pool/tree: Support pool for tree service
f5b23eb225 [#74] netmap/parser: Update antlr4 generator
70f23dd1ea [#74] pre-commit: Exclude auto-generated files
57f874048b [#74] go.mod: Update antlr4 dependency
a397d1fd15 [#74] go.mod: Update dependencies
Compare 10 commits »
dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#123 2023-06-05 10:50:42 +00:00
[#84] add tracing support

@pogpp It seems your last force push didn't change anything

dkirillov commented on pull request TrueCloudLab/frostfs-http-gw#56 2023-06-05 10:44:29 +00:00
Increase golangci-lint timeout

Why the commit message contains link to #53?

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#123 2023-06-05 07:42:46 +00:00
[#84] add tracing support

Actually, currently there are not default values for tracing

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#123 2023-06-02 12:37:40 +00:00
[#84] add tracing support

I still have some issues with pre-commit:

$ pre-commit run --all
...
trim trailing whitespace.................................................Failed
- hook id: trailing-whitesp…
dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#123 2023-06-02 11:45:07 +00:00
[#84] add tracing support

@pogpp Please run pre-commit run --all and fix issues

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#125 2023-06-01 14:58:13 +00:00
Fix gitlab intergation

Encrypted payload can be different from plain one. So we want to know the real size in HEAD requests for example, so we store real size in object attributes and tree node attribute

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#125 2023-06-01 14:54:34 +00:00
Fix gitlab intergation

Yes, aws presign generates the url like one that we couldn't handle in gitlab integration

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#123 2023-06-01 14:36:22 +00:00
[#84] add tracing support

Let's write REQUEST S3 instead of just REQUEST

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#123 2023-06-01 14:34:51 +00:00
[#84] add tracing support

It seems we can omit explicit ctx contextContext param and use r.Context()

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#120 2023-06-01 14:23:12 +00:00
Process 'object not found' error when object is missing in object service

though it's a matter of taste

dkirillov commented on pull request TrueCloudLab/frostfs-s3-gw#120 2023-06-01 14:20:44 +00:00
Process 'object not found' error when object is missing in object service

I mean instead of:

if client.IsErrObjectAlreadyRemoved(obj.Error) {
	// ...
	n.cache.DeleteObjectName(bkt.CID, bkt.Name, obj.Name)
}

if client.IsErrObjectNotFound(obj.Error)
0b82bd51d3 [#125] Update CHANGELOG.md
1381020300 [#125] Fix trailing whitespaces
843cd4c094 [#125] Handle negative Content-Length on put
089e2c8bc8 [#125] api/auth: DisableURIPathEscaping for presign
Compare 3 commits »
dkirillov created pull request TrueCloudLab/frostfs-s3-gw#125 2023-06-01 13:51:22 +00:00
WIP: Fix gitlag intergation
dkirillov created branch bugfix/content-length-presigned-url in dkirillov/frostfs-s3-gw 2023-06-01 13:49:15 +00:00
65cb6d7e07 [#XX] Fix trailing whitespaces
71afb87851 [#XX] Handle negative Content-Length on put
725ec428cd [#XX] api/auth: DisableURIPathEscaping for presign
81e860481d [#122] Fix linter warnings
e97fea30ea [#122] Update actions for forgejo ci
Compare 10 commits »