Router fix to handle url encoded form payload

[sravs-dev]

Fix for the issue #5512

When there is a POST request with payload type "application/x-www-form-urlencoded" , Input body is of type bytes. We need to convert it to string before processing

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ armab
✅ sravs-dev
❌ Sravanthi Konduru

---

Sravanthi Konduru seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[m4dcoder]

@sravs-dev Thanks for the contrib! Can you add a unit test at https://github.com/StackStorm/st2/blob/master/st2api/tests/unit/controllers/v1/test_base.py?

[sravs-dev]

@sravs-dev Thanks for the contrib! Can you add a unit test at https://github.com/StackStorm/st2/blob/master/st2api/tests/unit/controllers/v1/test_base.py?

Sure, will add

[sravs-dev]

@sravs-dev Thanks for the contrib! Can you add a unit test at https://github.com/StackStorm/st2/blob/master/st2api/tests/unit/controllers/v1/test_base.py?

Sure, will add

@m4dcoder Is there a setup guide that I can follow for running tests? I guess the tests expect mongo DB to be running in the localhost?

[sravs-dev]

@m4dcoder I see that my change is already covered in an existing test case here. I see that the unit test is temporarily disabled by @Kami in this commit . I have enabled the unit test as part of this PR. We can see if the existing test passes with my router fix.

[arm4b]

@sravs-dev Thanks for the fix!

From the CI issues (https://github.com/StackStorm/st2/runs/4706277178), only the lining is failing for test_sso.py, please reformat it with Black.

Could you please also update the CHANGELOG.rst with the description for this fix?

[sravs-dev]

@sravs-dev Thanks for the fix!

From the CI issues (https://github.com/StackStorm/st2/runs/4706277178), only the lining is failing for test_sso.py, please reformat it with Black.

Could you please also update the CHANGELOG.rst with the description for this fix?

@armab Thanks for checking, I fixed the format issue and updated changelog. Need approval to run CI checks.

[arm4b]

@sravs-dev Thanks! All good, the only missing is the CLA check. Looks like your github account wasn't associated with the commits.

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.

✅ sravs-dev
❌ Sravanthi Konduru

---

Sravanthi Konduru seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Could you try that by adding the email associated with those commits to your github account? See more details in the: #5513 (comment)

[sravs-dev]

@armab CLA check was failing due to wrong email associated with git commits. Corrected email and submitted new PR #5536 . Need approval in new PR to run CI checks.