TimSC / microcosm

OSM API v0.6 implemented in PHP
Other
8 stars 3 forks source link

no enough parameters to RequireAuth #6

Closed h4ck3rm1k3 closed 11 years ago

h4ck3rm1k3 commented 11 years ago

RequireAuth($user,$pass) takes two parms, I routed the params in from all the calls,

but how to pass them into GetTraceDetailsBackend from the callbacks? where are they stored? mike

h4ck3rm1k3 commented 11 years ago

https://github.com/h4ck3rm1k3/microcosm/commit/48cd5d72d9c5f6929ec0396b4337cd2a8f6e990a

h4ck3rm1k3 commented 11 years ago

Also called in web/import and web/export with no params, where do they come from? mike

TimSC commented 11 years ago

I overhauled the authentication functions. I also accidentally merged everything to my master branch and I can't be bothered to undo it. It contains some experimental code which might be a bit detrimental to performance but for small things it should be ok. Uploaded everything to my master branch. Let me know how it works out.

h4ck3rm1k3 commented 11 years ago

ok will do, give me a day pls.

h4ck3rm1k3 commented 11 years ago

Ok, now master compiles the php in hphp without errors except for the include to web/config.php but i dont think that is an error. congrats! static checking is done for now. mike

TimSC commented 11 years ago

Thanks for testing the code! It is much better for it.