Log message #4150976

# At Username Text
# Jun 13th 2018, 10:03 rudy1976s @lorenzo thank you
# Jun 13th 2018, 10:03 lorenzo not sure how to help you with that
# Jun 13th 2018, 10:02 lorenzo sounds like a problem with your web server not passing that data
# Jun 13th 2018, 10:02 johan_hedin pr($_GET);
# Jun 13th 2018, 10:02 johan_hedin pr($_POST); all are blank
# Jun 13th 2018, 10:02 johan_hedin pr($this->request->getQueryParams());
# Jun 13th 2018, 10:02 johan_hedin @lorenzo :
# Jun 13th 2018, 10:01 lorenzo the plugin needs to be loaded, part of the composer class map, the class needs the right namespace and so on
# Jun 13th 2018, 10:00 lorenzo as long as it can find the class, yes
# Jun 13th 2018, 09:58 rudy1976s because I set table name in Post model and the Table object generated looks for the standard underscored version and not my custom
# Jun 13th 2018, 09:57 rudy1976s The TableLocator is aware of the table name when using like the following ? $this->Posts = TableRegistry::getTableLocator()->get('MyPlugin.Postsl); ?
# Jun 13th 2018, 09:57 lorenzo do you see anthing in `$_GET` or `$_POST`?
# Jun 13th 2018, 09:49 johan_hedin when i am accessing https://apis.domain.com/fb-pages/auth/?foo=bar
# Jun 13th 2018, 09:47 johan_hedin hi there..why my cakephp always returns blank array with $this->request->getQueryParams() or $this->request->getQuery('foo')
# Jun 13th 2018, 09:25 rudy1976s @lorenzo open issue https://github.com/cakephp/cakephp/issues/12228
# Jun 13th 2018, 09:19 lorenzo sounds likely
# Jun 13th 2018, 09:19 neothermic @lorenzo mm, as noted, the email passes both normal and deep in my testing, leading me to conclude the issue is further up the chain (as in this data came in with some whitespace unicode that made it invalid)
# Jun 13th 2018, 09:13 lorenzo @neothermic `deep` is the argument to theck mx records
# Jun 13th 2018, 09:13 rudy1976s @lorenzo ok I will open a ticket
# Jun 13th 2018, 09:11 neothermic but I'm pondering if the original data had some weird characters before/after the address
# Jun 13th 2018, 09:11 neothermic and the mx check
# Jun 13th 2018, 09:11 neothermic I have no idea if we're doing deep or not, but it passes both regexes
# Jun 13th 2018, 09:10 lorenzo even when setting the `deep` option to false?
# Jun 13th 2018, 09:08 neothermic hmm. trying to work out why an email is failing validation, but when I run through the code by-hand (inc the regexes), it should pass :|
# Jun 13th 2018, 09:06 lorenzo no, I think we just check if the thing exists at all
# Jun 13th 2018, 09:06 neothermic but then we do nothing with it? :)
# Jun 13th 2018, 09:06 neothermic oooh, hah, it's byref
# Jun 13th 2018, 09:05 lorenzo @neothermic that’s just actually pure php fun
# Jun 13th 2018, 09:04 lorenzo @josbeir ok, let me know of your progress
# Jun 13th 2018, 09:04 neothermic https://github.com/cakephp/cakephp/blob/2.x/lib/Cake/Utility/Validation.php#L475 where does `$mxhosts` come from? it's not defined _anywhere else_
# Jun 13th 2018, 09:04 neothermic Hmm, I think I've found another fun weirdness in 2.x's codebase
# Jun 13th 2018, 09:00 josbeir i'll just go trough all bits and see whats the best way to fix it :slightly_smiling_face:
# Jun 13th 2018, 08:58 josbeir the whole validation => context is pretty complex, haven't looked at it in great detail
# Jun 13th 2018, 08:58 josbeir i'm not sure, thats the problem :)
# Jun 13th 2018, 08:57 josbeir for me yes, but dont think it should work like that
# Jun 13th 2018, 08:56 lorenzo would the result be desirable to you?
# Jun 13th 2018, 08:56 lorenzo what happens if you sacrifice one of the conflicting features?
# Jun 13th 2018, 08:55 lorenzo I’m aware of the ticket
# Jun 13th 2018, 08:53 josbeir (the issue is in the DocumentContext::error method that doesn't take care of nested paths)
# Jun 13th 2018, 08:52 josbeir hope i'm not to cryptic with the description :)
# Jun 13th 2018, 08:52 josbeir and i'm not sure how to 'fix' this correctly, that both use cases work correctly