just a heads-up, though this is probably more of a...
# helps
Sam Hulick
08/22/2021, 7:30 PMjust a heads-up, though this is probably more of a bug with CloudFormation: you can’t change route parameters, e.g.
GET /things/{id}GET /things/{thingId}Sam Hulick
08/22/2021, 7:30 PMI can see why it does this.. it doesn’t understand I’m modifying the route, it thinks I’m making a new one
f
Frank
08/22/2021, 7:51 PM
Thats right @Sam Hulick. It doesnt seem like the CFN team is going to fix this. So as a framework, SST should ideally detect this, removes the old route and then add a new one. What do you think @thdxr?
t
thdxr
08/22/2021, 7:52 PMCan we generate an ID for the construct that hashes the path? That way changes to it will create a new construct
thdxr
08/22/2021, 7:52 PMI haven't actually looked into how the Api construct works
f
Frank
08/22/2021, 8:00 PM
You mean creating a new API project replacing the old one?
a
Ashishkumar Pandey
08/23/2021, 2:42 AMYeah, I stumbled upon this a few months back, figured it was a Cfn problem so didn’t think much about it.
Ashishkumar Pandey
08/23/2021, 2:43 AMWhile it makes sense that sst could handle it internally, this improvement goes into debatable territory because we’re modifying Cfn behaviour from what is expected.