*** Serious vulnerability

[Forums]

The macro _must_ authenticate or otherwise by definition the data is available for anybody to view.

Tim

[icedlava]

The macro _must_ authenticate or otherwise by definition the data is available for anybody to view.

Tim

That's correct, but there are other ways to feed the macro most likely.

Is there any vulnerability reporting and fix process for webERP? I think it could be useful to have even though the use might occur rarely. Might help so we don't feed anyone that might be bored enough to want to go play with their new found knowledge.

[Forums]

Agreed

[icedlava]

More generally, and as per my prior post in this thread, this highlighted to me some other possible issues so I'll be looking at session.inc and ConnectDB especially in reference to the $_SESSION['DatabaseName'] and how it is used in the code, more generally. I think it is worth a review.

[jo lwebuga]

Thank you Tim and Icedlava for the warning

[phil]

Sorry my bad ... should be using the API for this function - I need to send an advice to the mailing list too.