Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Possibly corrupt comment table entries #96

Open
swagatam11 opened this issue Feb 24, 2020 · 0 comments
Open

Possibly corrupt comment table entries #96

swagatam11 opened this issue Feb 24, 2020 · 0 comments

Comments

@swagatam11
Copy link

I am using the GHTorrent data from Google BigQuery (mined in mid-2018) . The user comments table shows several cases where in a given month, the same developer makes the same comment in several different projects. Consider the following, randomly selected example:
github issue

Here, the developer with ID 10527 leaves a comment in 9 different projects in the same month. All of these comments are the following:

"
b'How about return api.MIN_VERSION < float(version) < api.MAX_VERSION ? !@#$%^&()3093794166 I don't think it works like this. If you're not passing in a string that was already wrapped in _(), it won't get picked up for translation. If you're passing in a string that was wrapped in _(), this is a no-op. !@#$%^&()3093794166 Please put spaces insde your moustaches. Here's the perl that might fix it: perl -pi -e 's/\ {\ {([^}]+)}}/{{ \ 1 }}/g !@#$%^&()3093794166 Why can't you use assertRedirects here? !@#$%^&()3093794166 Can you make that a manager method? Doing a filter in the view crosses my ill-defined MVC separation line. !@#$%^&()3093794166 I'm trying to keep all the redirects at the bottom of urlpatterns, because I'm assuming there's going to be a lot, and we don't really care about reading them once they work. I think redirects will move to a separate urls file eventually, or at least a se !@#$%^&()3093794166 Can't you get this with addon.compatible_apps? That one has better caching. !@#$%^&()3093794166 Same thing. Wrapping msg here doesn't do anything, AFAIK. !@#$%^&()3093794166 Won't this be an XSS hole unless you escape fallback first? !@#$%^&()3093794166 "If you put "" -- coding: utf-8 --"" at the top of the file, you can use real unicode characters in the source." !@#$%^&()3093794166 I didn't realize that wasn't happening already. Good catch. !@#$%^&()3093794166 Instead of setting this every time there's an error, why not do this at the bottom? if not all(status_summary.values()): status = 500 !@#$%^&()3093794166 Weird, I never saw any problems with this. I ran it locally and on preview. !@#$%^&*()3093794166 Can you generate this outside of settings? It seems like this is only useful in the urlresolver code.'
"

Is there some way to address this issue?

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

No branches or pull requests

1 participant