Collision issue on Samsung Galaxy S20 bundled as release

[totebo]

@britzl

If it is a hash then you can use it as a key in a table. The hashes will be unique even in a debug build. If it is a URL you need to convert it to a unique string from its components before using it as a key:

local EMPTY = hash("")

local function to_key(url)
	return hash_to_hex(url.socket or EMPTY) .. hash_to_hex(url.path or EMPTY) .. hash_to_hex(url.fragment or EMPTY)
end

This is what I’ve been missing. I modified the test project to create a key in this way and now it works!

Thanks for helping me find the missing piece of the puzzle.

@britzl

And now that I think of it maybe we should handle this under-the-hood somehow. We should provide a better __tostring meta-method for URLs. It’s not the first time someone has stumbled on to this issue.

The above fix might be obvious to some, but it’s black magic to me. Would love to have an easier way to do this.