[#223] Add store content language #240

r.loginov · 2023-10-16T17:20:49Z

r.loginov commented

2023-10-16 17:20:49 +00:00

Closes #223

Signed-off-by: Roman Loginov r.loginov@yadro.com

Closes #223 Signed-off-by: Roman Loginov <r.loginov@yadro.com>

r.loginov self-assigned this 2023-10-16 17:20:49 +00:00

dkirillov reviewed 2023-10-17 06:24:09 +00:00

api/layer/object.go Outdated

					
				@ -734,3 +734,3 @@

				func IsSystemHeader(key string) bool {

					_, ok := api.SystemMetadata[key]

					return ok || strings.HasPrefix(key, api.FrostFSSystemMetadataPrefix)

					return ok || strings.HasPrefix(key, api.FrostFSSystemMetadataPrefix) || key == api.ContentLanguage

dkirillov commented

2023-10-17 06:24:09 +00:00

Let's add api.ContentLanguage to api.SystemMetadata map

Let's add `api.ContentLanguage` to `api.SystemMetadata` map

👍 1

dkirillov marked this conversation as resolved

dkirillov approved these changes 2023-10-17 06:25:12 +00:00

dkirillov left a comment

It would be nice to have tests to check that we successfully store and retrieve content language header

👍 1

r.loginov force-pushed feature/223-add_store_content_language from 6672cbb82f to 91de5d8d3c

2023-10-17 09:04:49 +00:00

Compare

r.loginov requested review from storage-services-committers 2023-10-17 09:05:46 +00:00

r.loginov requested review from storage-services-developers 2023-10-17 09:05:47 +00:00

dkirillov reviewed 2023-10-17 11:27:19 +00:00

api/handler/multipart_upload_test.go Outdated

					
				@ -256,0 +262,4 @@

					partSize := 5 * 1024 * 1024

					exceptedContentLanguage := "en"

					headers := map[string]string{

						"Content-Language": exceptedContentLanguage,