token prop merged out about vue2leaflet

[mikeu]

Hi @depperm , is there a reason that passing the token as an option doesn't work for you? I think you should be able to use something along the lines of <LTileLayer :options="{ token: myTokenVal }"> to pass your value through to the constructor of the Leaflet TileLayer.

I'm also curious, I can't find token as a parameter in the Leaflet documentation, do you have a reference for where it's used, how it's defined, etc.?

[depperm]

that may work, I'll have to try that.

I didn't necessarily see token in Leaflet documentation, but seeing it in old vue-leaflet and reading about CSRF tokens I thought it was an added feature that fell by the wayside

[mikeu]

The change in #32 introduced the params prop to allow an arbitrary object to be passed to the Leaflet tileLayer constructor. For backwards compatibility with existing projects that had used :attribution and :token props, it left them as valid props and simply copied them into the params object before passing it on to Leaflet.

At that stage, there would have been no difference between <LTileLayer :token="myToken"> and <LTileLayer :options="{ token: myToken }">. Part of the refactor in #231 involved deprecating the former, but the latter is still supported and totally equivalent. So as far as I can tell, I don't think any change is needed here, unless you can confirm that there is a need to pass a token option to Leaflet, and that using the existing options object prop doesn't allow that—in which case, please do feel free to follow up and reopen this issue!