Will Kahn-Greene [:willkg] ET needinfo? me Reporter
	Description • 9 years ago

Saw a couple of these over the last month:

...
  File "/data/www/input.mozilla.org/input/vendor/src/django-rest-framework/rest_framework/serializers.py", line 546, in errors
    ret = self.from_native(data, files)
  File "/data/www/input.mozilla.org/input/vendor/src/django-rest-framework/rest_framework/serializers.py", line 374, in from_native
    attrs = self.perform_validation(attrs)
  File "/data/www/input.mozilla.org/input/vendor/src/django-rest-framework/rest_framework/serializers.py", line 307, in perform_validation
    attrs = validate_method(attrs, source)
  File "/data/www/input.mozilla.org/input/fjord/feedback/models.py", line 662, in validate_url
    '{0} is not a valid url'.format(value))
UnicodeEncodeError: 'ascii' codec can't encode character u'\xe4' in position 8: ordinal not in range(128)

Need to fix that so it handles urls correctly.

	Will Kahn-Greene [:willkg] ET needinfo? me Reporter
	Comment 1 • 9 years ago

Making this a P2 because it's worth fixing asap and I suspect it's a straight-forward fix.

	Will Kahn-Greene [:willkg] ET needinfo? me Reporter
	Comment 2 • 9 years ago

I talked to L Guruprasad about this on IRC and I think he agreed to take it, so I'm assigning it to him.

LGuruprasad: If you have questions or issues, let me know.

	Will Kahn-Greene [:willkg] ET needinfo? me Reporter
	Comment 3 • 9 years ago

In a PR: https://github.com/mozilla/fjord/pull/524

	Will Kahn-Greene [:willkg] ET needinfo? me Reporter
	Comment 4 • 9 years ago

Landed in master: https://github.com/mozilla/fjord/commit/7fa256a654e5a32dea4eb87857dc73b531656e62

Will push out next deploy.

	Will Kahn-Greene [:willkg] ET needinfo? me Reporter
	Comment 5 • 9 years ago

Pushed just now!