Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Smithy 1.27.2 #2226

Merged
merged 12 commits into from
Mar 14, 2023
Merged

Upgrade to Smithy 1.27.2 #2226

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
60d84d1
Upgrade to Smithy 1.27.1
rcoh Jan 18, 2023
34fd0a2
Fix bugs exposed by 1.27.1
rcoh Jan 18, 2023
16cff9b
Fix rules engine customization
rcoh Jan 19, 2023
59db199
update rfc3339 parsing test
rcoh Jan 19, 2023
48d5f1e
refactor timestamp format & remove upstreamed test
rcoh Jan 19, 2023
812e1bb
Update changelog
rcoh Jan 19, 2023
b919fca
rename test
rcoh Jan 19, 2023
e96790b
fix datetime parsing on server and clients
rcoh Jan 20, 2023
f59ad8d
fix up a few more failing tests
rcoh Feb 6, 2023
4b461cc
precommit lints
rcoh Mar 14, 2023
79a645c
Fix bad merge
rcoh Mar 14, 2023
45e5de4
Merge branch 'main' into smithy-1.27
rcoh Mar 14, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix rules engine customization
  • Loading branch information
@rcoh
rcoh committed Mar 14, 2023
commit 16cff9b43bb955150986bd9de456327b0be4f42e
28 changes: 14 additions & 14 deletions ...-codegen/src/main/kotlin/software/amazon/smithy/rustsdk/endpoints/AwsEndpointDecorator.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,8 +10,8 @@ import software.amazon.smithy.model.Model
import software.amazon.smithy.model.shapes.ServiceShape
import software.amazon.smithy.model.shapes.ShapeId
import software.amazon.smithy.model.transform.ModelTransformer
import software.amazon.smithy.rulesengine.language.EndpointRuleSet
import software.amazon.smithy.rulesengine.language.syntax.parameters.Builtins
import software.amazon.smithy.rulesengine.language.syntax.parameters.Parameter
import software.amazon.smithy.rulesengine.language.syntax.parameters.Parameters
import software.amazon.smithy.rulesengine.traits.EndpointRuleSetTrait
import software.amazon.smithy.rust.codegen.client.smithy.ClientCodegenContext
Expand Down Expand Up @@ -55,20 +55,18 @@ class AwsEndpointDecorator : ClientCodegenDecorator {
return ModelTransformer.create().mapTraits(model) { _, trait ->
when (trait) {
is EndpointRuleSetTrait -> {
val epRules = EndpointRuleSet.fromNode(trait.ruleSet)
val rules = trait.ruleSet.expectObjectNode()
val params = rules.expectObjectMember("parameters")
val newParameters = Parameters.builder()
epRules.parameters.toList()
.map { param ->
param.letIf(param.builtIn == Builtins.REGION.builtIn) { parameter ->
parameter.toBuilder().required(true).build()
}
params.members.map { (key, value) ->
val param = Parameter.fromNode(key, value.expectObjectNode())
param.letIf(param.builtIn == Builtins.REGION.builtIn) { parameter ->
parameter.toBuilder().required(true).build()
}
.forEach(newParameters::addParameter)

val newTrait = epRules.toBuilder().parameters(
newParameters.build(),
).build()
EndpointRuleSetTrait.builder().ruleSet(newTrait.toNode()).build()
}.forEach(newParameters::addParameter)
EndpointRuleSetTrait.builder()
.ruleSet(rules.toBuilder().withMember("parameters", newParameters.build().toNode()).build())
.build()
}

else -> trait
Expand Down Expand Up @@ -244,7 +242,9 @@ class AwsEndpointDecorator : ClientCodegenDecorator {
}

ServiceConfig.ConfigStruct -> rust("endpoint_url: Option<String>,")
else -> {}
ServiceConfig.ConfigStructAdditionalDocs -> emptySection
ServiceConfig.Extras -> emptySection
else -> emptySection
}
}
}
Expand Down