| # |
Jun 12th 2018, 14:16 |
 josbeir |
they both check the header tho... sure the 'X-CSRF-Token' header is present with the token ? |
| # |
Jun 12th 2018, 14:16 |
josbeir |
is it from the CsrfComponent or middleware? |
| # |
Jun 12th 2018, 14:16 |
josbeir |
LubosR: did you check where the exception is thrown |
| # |
Jun 12th 2018, 14:14 |
admad |
and the loadModel() call remains same |
| # |
Jun 12th 2018, 14:14 |
neon1024 |
Thanks |
| # |
Jun 12th 2018, 14:14 |
admad |
@neon1024 `$this->modelFactory('Table', [new EndpointLocator(), 'get']);` should do it |
| # |
Jun 12th 2018, 14:10 |
admad |
ok, gtg, ttyl |
| # |
Jun 12th 2018, 14:09 |
neon1024 |
@admad Okay, I will submit a pull request to update the readme |
| # |
Jun 12th 2018, 14:09 |
LubosR |
I can see csfrToken is in request header cookie too |
| # |
Jun 12th 2018, 14:09 |
admad |
you just need to change the line `$this->modelFactory('Endpoint', ['Muffin\Webservice\Model\EndpointRegistry', 'get']);` to use the new locator instead of EndpointRegisry |
| # |
Jun 12th 2018, 14:07 |
neon1024 |
Ok |
| # |
Jun 12th 2018, 14:07 |
admad |
@neon1024 checkout the Controller::loadModel() code |
| # |
Jun 12th 2018, 14:07 |
LubosR |
josebeir I have noticed that there is no Csfr at Application.php |
| # |
Jun 12th 2018, 14:07 |
admad |
no, you can configure the locator to use for specific model types |
| # |
Jun 12th 2018, 14:07 |
LubosR |
admad, josbeir... i am sending now X-CSRFToken header in XMLHttpRequest, but still getting CSFR token mistmatch exception, any more ideas what I am doing wrong? |
| # |
Jun 12th 2018, 14:06 |
neon1024 |
I will try and find some time to test out the RC :slightly_smiling_face: |
| # |
Jun 12th 2018, 14:06 |
neon1024 |
Yeah, but `loadModel` relies on the Registry doesn’t it? Which has been deprecated |
| # |
Jun 12th 2018, 14:05 |
neon1024 |
Woah, look how much code marlinc has contributed! :O |
| # |
Jun 12th 2018, 14:05 |
admad |
hmm actually one should still be able to use ->loadModel() |
| # |
Jun 12th 2018, 14:04 |
neon1024 |
:thumbsup: |
| # |
Jun 12th 2018, 14:04 |
admad |
looks fine |
| # |
Jun 12th 2018, 14:04 |
neon1024 |
https://github.com/UseMuffin/Webservice/pull/71/files#diff-04c6e90faac2675aa89e2176d2eec7d8R136 |
| # |
Jun 12th 2018, 14:03 |
admad |
make sure that cake is red velvet ;) |
| # |
Jun 12th 2018, 14:03 |
neon1024 |
@admad Just as a related aside, that usage of the Locator classes is correct is it? |
| # |
Jun 12th 2018, 14:03 |
neon1024 |
The one where I get the shiny make-stuff-happen buttons :zany_face: |
| # |
Jun 12th 2018, 14:02 |
neon1024 |
Oh right, er, member? |
| # |
Jun 12th 2018, 14:01 |
slackebot |
Action: josbeir loves bad jokes |
| # |
Jun 12th 2018, 14:01 |
josbeir |
i grinned |
| # |
Jun 12th 2018, 14:01 |
slackebot |
Action: admad slaps neothermic for this bad joke |
| # |
Jun 12th 2018, 14:00 |
neothermic |
woo, cake2 RC1!. Wait, what year is it? |
| # |
Jun 12th 2018, 14:00 |
admad |
merged and 2.0.0-RC1 released |
| # |
Jun 12th 2018, 13:57 |
admad |
you are already a contributor :) |
| # |
Jun 12th 2018, 13:56 |
neon1024 |
But only once approved by @admad :P |
| # |
Jun 12th 2018, 13:55 |
neon1024 |
..and on that day, I shall eat cake. |
| # |
Jun 12th 2018, 13:55 |
neon1024 |
One day I will earn contributor rights! |
| # |
Jun 12th 2018, 13:55 |
slackebot |
Action: neon1024 quickly checks the pr hasn’t already been merged |
| # |
Jun 12th 2018, 13:54 |
slackebot |
Action: neon1024 nods |
| # |
Jun 12th 2018, 13:54 |
neon1024 |
Like doing the Webservices plugin release? |
| # |
Jun 12th 2018, 13:48 |
admad |
have bigger fishes to fry :slightly_smiling_face: |
| # |
Jun 12th 2018, 13:48 |
josbeir |
its one of those.. hmm to core or not to core |
| # |
Jun 12th 2018, 13:48 |
josbeir |
:) |