# |
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 |
:) |
# |
Jun 12th 2018, 13:47 |
admad |
one time i wanted to add a support for `Html->meta('csrf')` to generate the tag @josbeir showEd but lazied out |
# |
Jun 12th 2018, 13:46 |
slackebot |
Action: josbeir defeated |
# |
Jun 12th 2018, 13:46 |
LubosR |
admad :D |
# |
Jun 12th 2018, 13:46 |
josbeir |
:,( |
# |
Jun 12th 2018, 13:46 |
neon1024 |
Such modesty! |
# |
Jun 12th 2018, 13:46 |
admad |
but i am better of the best |
# |
Jun 12th 2018, 13:45 |
admad |
its as safe as having a hidden form field with the token, which is what form helper does |
# |
Jun 12th 2018, 13:45 |
LubosR |
josbeir, admad: you two are the best, thanks. going to give it a try |