404 on "Mark this as solved" in question response emails

RESOLVED WORKSFORME

Status

P1
normal
RESOLVED WORKSFORME
5 years ago
2 years ago

People

(Reporter: mythmon, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

5 years ago
As a test, I just asked and answered a question. This is the url in the email for "Mark this problem as solved": 

https://support.allizom.org/en-US/questions/972883/solve/48481%205?watch=wrULRRfFcM

Notice the %20 before the ? in the url, which is space, which causes a 404. This is probably big deal, because it is stopping people from marking questions as solved, and probably discouraging them from returning to the question.
Not sure right now, but when I was testing this the last time, it was only on staging.
Very weird. I see this in the email template:

    {% trans url='http://' + host + solution_url %}
      If this answer solves your problem, please
      <br/><br/>
      <a class="button" style="width: 200px;" href="{{ url }}">Mark it as solved</a>
    {% endtrans %}


solution_url comes from:

    c['solution_url'] = self.answer.get_solution_url(watch=w[0])


get_solution_url() is:

    def get_solution_url(self, watch):
        url = reverse('questions.solve',
                      kwargs={'question_id': self.question_id,
                              'answer_id': self.id})
        return urlparams(url, watch=watch.secret)


I don't see anything that would cause that %20 to appear.
Also, the answer id in this case is: 484815 (https://support.allizom.org/en-US/questions/972883#answer-484815).

No idea how that can turn into 48481%205 ???
Is it possible the line is long and is getting word-wrapped or something like that? If that were true, I'm not sure why we're not seeing it elsewhere. But that's the only thing I can think of that might insert a space somewhere.
Hi,
I've been testing the bug but even faking the question_id and the id of the answer to match 972883 and 484815 i didn't reproduce the bug, i also created some tests on allizom and they work so i question if it's been already fixed
This is most likely resolved at this point. Haven't seen complaints from users.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.