Raise error in proper places

[theSage21]

It might be a good idea to use abort(404, <reason>) in some places like

• https://github.com/PyJaipur/PyJudge/blob/e3debe1e15447a6faac9d509e7ad5283731fa2c0/server.py#L89
• https://github.com/PyJaipur/PyJudge/blob/e3debe1e15447a6faac9d509e7ad5283731fa2c0/server.py#L102

some places however don't require the error. For example this

• https://github.com/PyJaipur/PyJudge/blob/e3debe1e15447a6faac9d509e7ad5283731fa2c0/server.py#L104

does not require an error.

[soumilk]

I would like to take this issue, basically in this, instead of a statement "contest does not exist", we should raise an abort 404 page error ?

[theSage21]

that's the basic idea. you will need to find other places in the code where this has to be done.

[soumilk]

ohk ... i am on it

[soumilk]

I have gone through the server.py and it seems that it has raised abort(404) at certain places already, couldn't find places which require error. Help required . And I thought to add a cool looking error page 404. Have a look, we can add it to the project.

[theSage21]

well, there are a few places where we are returning the string "contest inactive" or something like that. I was thinking if we could turn that into these pages it would be nice.

As for the 404 page it does look cool :heart_eyes: but you'll need to accomodate ALL error codes and not just 404 since that template gets called for ALL errors