remove base64.lua

[naturallymitchell]

https://github.com/foundpatterns/lighttouch-base/blob/6c2107c32bf6db4149d64f1874c6ca865d05f09d/init.lua#L19

if it's in use anywhere, it can be replaced by torchbear's base64 functions

[naturallymitchell]

https://github.com/foundpatterns/lighttouch-base/commit/58a9133645303dfb790b0e3153070cbe1c3e11ab

[Arnaz87]

I think it's being used in http authentication

[naturallymitchell]

yea, but that commit I referenced doesn't even seem to put it to use if so, could you switch it to the bindings' function

[Arnaz87]

But it is being used, I don't know what that commit is but you can look at the source code and see it required: init.lua line 19

But if there is a binding function, we should use that one instead

[naturallymitchell]

that it's required isn't important I don't see anywhere it's even used when you added it, you didn't even seem to make use of it the generate auth function just uses a math function

[Arnaz87]

I obviously added it because I needed it, if it's not being used in lighttouch-base it's in some package, I don't know where, but I was countering your point that that commit removed the require because it is being required currently, and so it may be used.

[Arnaz87]

And anyway I already agreed with you about removing it and using the binding's function

[naturallymitchell]

https://github.com/foundpatterns/lighttouch-html-interface/blob/10c9f6510492a13316a233cd873e1303dd5efd85/actions/generate_auth.lua#L35

[Arnaz87]

Oh, there is no binding function yet? I was confusing it with heck functions in the other issue. In that case this is unnecessary, the library is not doing any harm

[naturallymitchell]

Oh, there is no binding function yet?

you misread

[naturallymitchell]

this is now breaking it