Skip to content

feat(rpc): init#37

Merged
TorinAsakura merged 46 commits intomasterfrom
feat/files-engine
Apr 9, 2025
Merged

feat(rpc): init#37
TorinAsakura merged 46 commits intomasterfrom
feat/files-engine

Conversation

@TorinAsakura
Copy link
Member

#36

Возникли проблемы с публикацией пакетам на buf.build, пока залочен

@TorinAsakura TorinAsakura self-assigned this Feb 28, 2025
@TorinAsakura TorinAsakura marked this pull request as draft February 28, 2025 23:18
@Amiditin
Copy link
Member

Amiditin commented Apr 3, 2025

@TorinAsakura чеки прошли, что-то ещё нужно?

@TorinAsakura
Copy link
Member Author

@TorinAsakura чеки прошли, что-то ещё нужно?

ну, видимо запустить и проверить))) Приложить пруфы, что бакет рабочий, а ты как хотел?) Поправил чеки и всё?)) А пользоваться ты этим как потом будешь? Или ты забыл нафига я вообще это писал?) У нас риз работает с файлами почти везде, почти data driven development

@Amiditin
Copy link
Member

Amiditin commented Apr 5, 2025

Ветка https://github.com/atls/services/tree/feat/files-docker-setup

Запросы в gcs:

Details

files-service-gcs.webm


Запросы в minio не работают корректно, если мы прокидываем endpoint как S3_ENDPOINT=http://minio:9000, то после createUpload полученный url имеет вид http://minio:9000/public/c66c4870-fd40-4bff-84f1-eb381a465a00.png?..., а запрос на http://localhost:9000/public/... возвращает ошибку:

<?xml version="1.0" encoding="UTF-8"?>
<Error>
    <Code>SignatureDoesNotMatch</Code>
    <Message>The request signature we calculated does not match the signature you provided. Check your key and signing method.</Message>
    <Key>c66c4870-fd40-4bff-84f1-eb381a465a00.png</Key>
    <BucketName>public</BucketName>
    <Resource>/public/c66c4870-fd40-4bff-84f1-eb381a465a00.png</Resource>
    <RequestId>18334A44C0A43F9D</RequestId>
    <HostId>dd9025bab4ad464b049177c95eb6ebf374d3b3fd1af9251148b658df7ac2e3e8</HostId>
</Error>

Если поставить S3_ENDPOINT=http://localhost:9000, то ошибки с загрузкой файла в minio не будет, но confirmUpload упадёт с ошибкой:

{"level":50,"name":"S3FilesStorageAdapterImpl","body":{"errno":-111,"code":"ECONNREFUSED","syscall":"connect","address":"127.0.0.1","port":9000,"$metadata":{"attempts":3,"totalRetryDelay":100}},"severityNumber":17,"severityText":"ERROR","timestamp":1743820685936}

@TorinAsakura
Copy link
Member Author

Ветка https://github.com/atls/services/tree/feat/files-docker-setup

Запросы в gcs:

Details
Запросы в minio не работают корректно, если мы прокидываем endpoint как S3_ENDPOINT=http://minio:9000, то после createUpload полученный url имеет вид http://minio:9000/public/c66c4870-fd40-4bff-84f1-eb381a465a00.png?..., а запрос на http://localhost:9000/public/... возвращает ошибку:

<?xml version="1.0" encoding="UTF-8"?>
<Error>
    <Code>SignatureDoesNotMatch</Code>
    <Message>The request signature we calculated does not match the signature you provided. Check your key and signing method.</Message>
    <Key>c66c4870-fd40-4bff-84f1-eb381a465a00.png</Key>
    <BucketName>public</BucketName>
    <Resource>/public/c66c4870-fd40-4bff-84f1-eb381a465a00.png</Resource>
    <RequestId>18334A44C0A43F9D</RequestId>
    <HostId>dd9025bab4ad464b049177c95eb6ebf374d3b3fd1af9251148b658df7ac2e3e8</HostId>
</Error>

Если поставить S3_ENDPOINT=http://localhost:9000, то ошибки с загрузкой файла в minio не будет, но confirmUpload упадёт с ошибкой:

{"level":50,"name":"S3FilesStorageAdapterImpl","body":{"errno":-111,"code":"ECONNREFUSED","syscall":"connect","address":"127.0.0.1","port":9000,"$metadata":{"attempts":3,"totalRetryDelay":100}},"severityNumber":17,"severityText":"ERROR","timestamp":1743820685936}

ну так читай содержимое ошибки, там всё абсолютно понятно

сервис работает, просто ты работаешь с ним не так как он ожидает

в гпт загони в конце-то концов

@Amiditin Amiditin self-requested a review April 9, 2025 19:26
@TorinAsakura TorinAsakura merged commit 0916ab8 into master Apr 9, 2025
6 checks passed
@TorinAsakura TorinAsakura deleted the feat/files-engine branch April 9, 2025 19:33
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.

[Feature] Files Service v2 Рефактор тестов на node.js

2 participants