C2DEVEL-14695: return support of the backup service #34
Conversation
aws/corehandlers/handlers.go
Outdated
| if r.ClientInfo.SigningRegion == "" && aws.StringValue(r.Config.Region) == "" { | ||
| r.Error = aws.ErrMissingRegion | ||
| } else if r.ClientInfo.Endpoint == "" { | ||
| // if r.ClientInfo.SigningRegion == "" && aws.StringValue(r.Config.Region) == "" { |
There was a problem hiding this comment.
Не очень понятно зачем этот кусок кода закомментирован, а в истории коммитов и в задаче нет информаци про это изменение. Вынеси его в отдельный коммит пожалуйста и где-нибудь опиши
There was a problem hiding this comment.
В отдельный коммит вынес. Изначально убирал здесь проверку для возможности проставления пустого региона в конфиге провайдера. В целом, если будем работать с пустыми регионами только на уровне arn в теле запроса, можно будет откатить эти изменения.
There was a problem hiding this comment.
Пустой регион использовать не получается, тк есть датасорсы, которые используют значение региона как id, так что эту валидацию я бы все таки сохранила.
RudeZwloki7
force-pushed
the
C2DEVEL-14695
branch
from
58f69fa
to
875864a
Compare
| @@ -1230,7 +1240,8 @@ | |||
| "type":"structure", | |||
| "required":[ | |||
| "SelectionName", | |||
| "IamRoleArn" | |||
| "IamRoleArn", | |||
| "ResourceArns" | |||
There was a problem hiding this comment.
Кажется тут должно быть Resources
Based on #33
Changes: