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

Buggy newlines in certain 404 pages #6

Open
Parckwart opened this issue Oct 15, 2017 · 5 comments
Open

Buggy newlines in certain 404 pages #6

Parckwart opened this issue Oct 15, 2017 · 5 comments

Comments

@Parckwart
Copy link

http://zlal32teyptf4tvi.onion/path/foobar

If a search for a path is performed which results in no matches being found, the 404 message has "\n" all over the place.

@zaranmd
Copy link

zaranmd commented Oct 15, 2017

@Parckwart Do you mean that the results of page table are 404 not found?

@Parckwart
Copy link
Author

Whoops, I screwed up the link: Fixed link There is a 404 error page, as there should be. But there's a lot of "\n"'s in there red error message.

Screenshot:
zlal404

@zaranmd
Copy link

zaranmd commented Oct 15, 2017

@Parckwart i think it should be a temporarily problem... is that so important?
I have a question from you, did you run this project successfully?

@Parckwart
Copy link
Author

Sure, not an important issue at all. Just something I wanted to note.

I didn't attempt to set up the project (however I might take a look at it in the future). I Just noticed it with your installation.

@L3houx
Copy link

L3houx commented Mar 28, 2018

Hey @Parckwart , I fixed it! It was in the app.py script. The problem came when it called the error.html page. The ,404 at the end of the lane cause the \n. If you remove it, it will be all good. It had another problem link to that, if you searched something wrong, it will redirect to the error.html page but if you looked the HTTP code, this will be a 302 Found instead of 404 Not Found but the way he coded it, it can need huge modifications to change the 304 to 404. So if you removed , 404 it will work.

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

3 participants