Request parameters encoding doesn't work in v1 of the spec

[erikpragt-connectid]

Hi @deki ,

We found another issue, which is related the encoding of parameters. For example, when we send a timestamp, we could send the timestamp either as time=1970-01-01T00:00:00.004Z or time=1970-01-01T00%3A00%3A00.004Z. When we send this through a Function URL (so, v2 of the spec), all works well, we can call our endpoint using an encoded an un-encoded value.

However, when we use the ALB, which used v1, the unencoded version works, but when we encode the parameters, there's an exception in the code:

.w.s.m.s.DefaultHandlerExceptionResolver : Resolved [org.springframework.web.method.annotation.MethodArgumentTypeMismatchException: Method parameter 'time': Failed to convert value of type 'java.lang.String' to required type 'java.time.Instant'; Failed to convert from type [java.lang.String] to type [@org.springframework.web.bind.annotation.RequestParam java.time.Instant] for value [1970-01-01T00%3A00%3A00.904Z]]

I have a demo application if you need one, it doesn't do much, except for consuming a request parameter:

    @GetMapping
    public Instant time(@RequestParam Instant time) {
        return time;
    }

[erikpragt-connectid]

Hi there, I was wondering if there's anything we can do to resolve this? Currently this is blocking our testing and release process, and if there's anything we can do to help in this, please let us know.

[deki]

Hi @erikpragt-connectid,
sorry for the delay. You can work with your account manager or account solutions architect to prioritize this and get another engineer assigned (if you don't know your account team yet, use the form https://aws.amazon.com/contact-us/sales-support/). Alternatively if you have Enterprise Support you can also raise a case via: https://console.aws.amazon.com/support/home#/case/create?issueType=technical

[erikpragt-connectid]

Hi @deki , thanks, I'll do that!

[mbfreder]

Hi, @erikpragt-connectid, thanks for raisong the issue. I'll be working on it this sprint. I'll keep you updated on the progress.

[mbfreder]

Sorry for the delay, currently working on the issue. I should send a PR by end of day.

[mbfreder]

PR out.