yandex-cloud / docs

Yandex Cloud documentation
https://cloud.yandex.ru/docs
Creative Commons Attribution 4.0 International
209 stars 602 forks source link

fix(ru/en): fixed function java serveltet todo #752

Closed DKARAGODIN closed 3 weeks ago

DKARAGODIN commented 2 months ago

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=ru

Description of changes: Fixed API Gateway example in Servlet TODO tutorial. Without specifying service_account_id, functions return 403 Error.

There is one more difference I applied when I followed tutorial. My service account has three roles: functions.functionInvoker viewer editor

But It might not be necessary.

yfm-team commented 2 months ago

Все готово! Commit: 4ece17c24c8d31fa4a8b55b528d1da2e18365c34 Проверьте результат: RU. Check the result: EN.

myusosnovskay commented 1 month ago

Здравствуйте!

В практическом руководстве указано, что функции должны быть публичными, поэтому в текущем виде оно будет работать и без предложенных вами правок.

Вам было бы интересно самостоятельно доработать руководство, добавив IAM-авторизацию? Если нет, мы доработаем руководство сами и внесем ваши правки в рамках этой доработки.

DKARAGODIN commented 1 month ago

Да, я вас понял.

Я бы хотел доработать, только для этого нужно понять где хостится код примера. Вот тут я уже задавал этот вопрос:

https://github.com/yandex-cloud/docs/issues/762

myusosnovskay commented 1 month ago

Положили код примера в репозиторий: https://github.com/yandex-cloud-examples/yc-serverless-servlet/tree/main.

DKARAGODIN commented 3 weeks ago

Этот PR можно закрывать. PR на тему которую здесь обсудили сейчас подготовлю.