Add SameSite and Secure cookie options to script output

Thanks for the effort with this one @phamnguyen and sorry for taking so long with the reply!
I hadn't noticed your work with the issue before.

I added a few comments to the merge request.

Here are a couple of thoughts too:

• as the 1.4.x branch just changed today, there seems to be some conflicts that must be resolved
• I appreciate you did the extra work with changing some parts of the code outside the scope of your issue. I think it would still be best to keep the merge request as simple as possible at this point. We can create a new issue where we can clean up the code in general, if it seems necessary. So at this point I suggest those parts to be removed from the merge request which don't seem to be related to the issue in hand.
• Could you add an update hook for your new boolean settings?
• Could you add a bit more helpful description texts in the settings form? I mean, maybe mention is it safe to turn those new settings on, or can there be some issues in some cases when the settings are enabled?

Thank you!

Thanks for all the changes you did to the code @phamnguyen !
I think it looks good now. I tested your changes and everything seems to work.

Even though I previously said we don't want to touch unrelated parts of the module within this issue, I still kind of had to do a small change to the update hooks. They were previously added to the wrong file (.module) and you added the new hook to the correct place (.install file) so I just moved all the old hooks to the .install file, reorganized them a bit and renamed your hook so it should be consistent with the old ones.

I also liked the description texts you had added to the form, great job with those too.

I'll move this to RTBC

This should be now added to the latest release 1.4.2

Thanks for your contributions @phamnguyen !