Skip to content

Fixed #30405 -- Fixed source code mismatch crash in ExceptionReporter. #11886

Merged
felixxm merged 2 commits intodjango:masterfrom
hramezani:ticket_30405
Nov 12, 2019
Merged

Fixed #30405 -- Fixed source code mismatch crash in ExceptionReporter. #11886
felixxm merged 2 commits intodjango:masterfrom
hramezani:ticket_30405

Conversation

@hramezani
Copy link
Copy Markdown
Member

@felixxm felixxm requested a review from apollo13 October 11, 2019 12:42
@apollo13
Copy link
Copy Markdown
Member

@felixxm LGTM, there might still be cases where the lines are wrong, but at least this should prevent errors.

@felixxm felixxm self-assigned this Nov 12, 2019
@felixxm felixxm changed the title Fixed #30405 -- IndexError in _get_lines_from_file when module does not match file contents (via loader). Fixed #30405 -- Fixed source code mismatch crash in ExceptionReporter. Nov 12, 2019
@felixxm
Copy link
Copy Markdown
Member

felixxm commented Nov 12, 2019

@apollo13 Thanks for the review ⭐
@hramezani Thanks for this patch 👍

@felixxm felixxm merged commit 4b78546 into django:master Nov 12, 2019
@akx
Copy link
Copy Markdown
Contributor

akx commented Nov 13, 2019

👍, nice to see this merged. I was seeing the same issue with Jinja templates (using django_jinja – there's niwinz/django-jinja#238 over there).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants