Skip to content

Conversation

@gnadelwartz
Copy link
Contributor

@gnadelwartz gnadelwartz commented Mar 16, 2018

What's in this PR:

  • simple error handler to display a nicer error message as the default simple one liner
  • change to miniserv.pl to call error handler with real return code (e.g. 403), but return 200 to browser
  • to statisfy PCI Scanner the error handler has to be activated in miniserv.conf with error_handler_403=error403.cgi

What's not in this PR:

  • activatvation of error handler
  • configurable return code provided to browser
  • no UI to manage error handler

@gnadelwartz
Copy link
Contributor Author

gnadelwartz commented Mar 23, 2018

how will we proceed with this topic? try to stisfy PCI scnanner or don't care?

@jcameron
Copy link
Collaborator

How does this patch resolve the problem? I'm not seeing how that custom error handler is used?

@gnadelwartz
Copy link
Contributor Author

as stated in the issue I modify the http return code it custom handlers are used and a simple custom handler.

curently the handler has to be activated by hand, so what you possibly missing is UI support for activation?

@cooperdjm
Copy link

So sorry for my ignorance, I am also experiencing the false flag PCI failure. Can someone point me in the direction of how to activate the handler by hand? I am running Webmin 1.941

@cooperdjm
Copy link

So sorry for my ignorance, I am also experiencing the false flag PCI failure. Can someone point me in the direction of how to activate the handler by hand? I am running Webmin 1.941

I think I figured it out, well no, let's be honest I read some more and saw there are instructions to modify the .pl file.

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.

3 participants