Unable to grant custom badge for non-empty reason field

Hi, I checked latest both stable 1.9.3+2 and test-passed and realized that custom badge cannot be granted to user if the reason field is not empty.

Steps to reproduce:

  1. create a new simple badge using font-awasome icon, without SQL
  2. go to admin/users, find desirable user
  3. click Edit badges
  4. select the badge created
  5. type a reason (important!) https://www.cuba-platform.com/discuss/t/redirecting-http-to-https/3977/4
  6. click Grant
  7. you’ll get an error, see details below in the picture.

If the reason field is empty, there is no error.

My forum is in a subfolder, probably that’s a reason of the fail:

2 Likes

This looks incredibly likely to be the reason, unable to repro on non-subfolder.

4 Likes

Hi,
I was able to find the issue. I found it out after seeing that we were returning 404 in the first screenshot instead of 400. We were not taking care of relative_root_url while parsing reason of badge.

I have created PR for the same here:
https://github.com/discourse/discourse/pull/6836

7 Likes

This has been merged. Thank you @saurabhp!

4 Likes

This topic was automatically closed after 3 days. New replies are no longer allowed.