# |
Dec 15th 2017, 15:09 |
hmic |
the request does to the uri including the sub folder, of course |
# |
Dec 15th 2017, 15:09 |
hmic |
which is wrong all the time, isnt it? |
# |
Dec 15th 2017, 15:05 |
admad |
but that only matters if you app is in a sub folder |
# |
Dec 15th 2017, 15:04 |
admad |
only different is here() had the option to return path with or without "base" while the latter only returns without base. |
# |
Dec 15th 2017, 15:03 |
admad |
here() included query string and so does getRequestTarget() |
# |
Dec 15th 2017, 15:01 |
inoas |
it always returned the full path (without the query string I believe) |
# |
Dec 15th 2017, 15:01 |
neon1024 |
I don’t think the functionality exists in the framework |
# |
Dec 15th 2017, 15:01 |
inoas |
admad well a big red box should be okay, I am just not certain that I really know how to implement the feature (in Cakephp3/Middleware/Formhelper Code Stack) |
# |
Dec 15th 2017, 15:01 |
neon1024 |
Indeed not |
# |
Dec 15th 2017, 15:00 |
admad |
@neon1024 $request->here() never returned domain name either |
# |
Dec 15th 2017, 14:16 |
inoas |
yes I understand |
# |
Dec 15th 2017, 14:16 |
admad |
we used to get lot of tickets because of it |
# |
Dec 15th 2017, 14:15 |
admad |
inoas: they were removed on purpose as they caused lot of problem. for e.g you load a page with form in one page, then load another page in another tab then the token in 1st become invalid |
# |
Dec 15th 2017, 12:56 |
inoas |
it worked for the most part to stop bots from spamming contact forms |
# |
Dec 15th 2017, 12:55 |
inoas |
before I open a ticket about it... sorry for highlighting... admad did it get removed on purpose or just not reimplemented @ https://book.cakephp.org/2.0/en/core-libraries/components/security-component.html#SecurityComponent::$csrfUseOnce ? |
# |
Dec 15th 2017, 12:53 |
inoas |
aka csrfUseOnce = true |
# |
Dec 15th 2017, 12:51 |
inoas |
There is no way to do one-time tokens with CSRF Middleware, right? |
# |
Dec 15th 2017, 12:47 |
inoas |
hmic yeah one thing is complying with some PSR, the other thing is removing really useful stuff ... bad |
# |
Dec 15th 2017, 12:47 |
inoas |
so kudos |
# |
Dec 15th 2017, 12:47 |
inoas |
it works with both: cells and with view extending... whereas CsrFComponent has issues |
# |
Dec 15th 2017, 12:46 |
inoas |
whoever did the new CsrfMiddleware... |
# |
Dec 15th 2017, 12:41 |
hmic |
if not, i'd rather resort to the _SERVER global to get the info, instead of using this combersome thing :( |
# |
Dec 15th 2017, 12:41 |
hmic |
is there a note on the removed here() to what to use instead?! |
# |
Dec 15th 2017, 12:40 |
hmic |
that lookf fucking ugly! |
# |
Dec 15th 2017, 12:39 |
neon1024 |
Dirtbag, but it’ll work `'dl' => $request->getUri()->getScheme() . '://' . $request->getUri()->getHost() . $request->getUri()->getPath() . '?' . $request->getUri()->getQuery()` |
# |
Dec 15th 2017, 12:29 |
neon1024 |
As I need the DL, https://developers.google.com/analytics/devguides/collection/protocol/v1/parameters#dl |
# |
Dec 15th 2017, 12:20 |
neon1024 |
As I need the domain as well, the old `Router::url($urlArray, true)` |
# |
Dec 15th 2017, 12:20 |
neon1024 |
So if here() has been deprecated in favour of $request->getRequestTarget() how do we get the complete url for the current request? Have to build it using the `$request->getUri()` parts? |
# |
Dec 15th 2017, 12:12 |
neon1024 |
Oh, it’s only implemented in the class instance and not in the interface, hmm, might need to refactor my typehint |
# |
Dec 15th 2017, 12:11 |
neon1024 |
Ah it does :slightly_smiling_face: `$request->clientIp()` :) |
# |
Dec 15th 2017, 12:11 |
neon1024 |
Does the Cake request object store the remote ip? |
# |
Dec 15th 2017, 11:22 |
ono-t |
weekend is just around the corner! |
# |
Dec 15th 2017, 11:11 |
meowcate |
yay |
# |
Dec 15th 2017, 11:06 |
neon1024 |
Happy friday! |
# |
Dec 15th 2017, 11:06 |
neon1024 |
Morning everyone |
# |
Dec 15th 2017, 10:38 |
meowcate |
@Ayushi Can you be more specific ? |
# |
Dec 15th 2017, 10:36 |
Ayushi |
i want to save record in a table update another table at once |
# |
Dec 15th 2017, 10:35 |
Ayushi |
i need help |
# |
Dec 15th 2017, 10:35 |
meowcate |
Happy to help :slightly_smiling_face: |
# |
Dec 15th 2017, 10:35 |
Ayushi |
hello |
# |
Dec 15th 2017, 10:29 |
justcharlz |
@meowcate, thank you very much for the observation. Corrected and working like ...(lost word to use here)... thank you |