You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
For some reason the Server header is encoded in hex. If it was in just plain text we could grep for the string and change it.
Improvement
Put the cleartext server header in the code instead of the hex representation because it's running on the server anyways.
Also, what was the reason to use base16 on it? It's unlikely anyone would run this on the target system with Python installed, so obfuscation isn't really necessary.
The text was updated successfully, but these errors were encountered:
Hi Vincent, that Server header was encoded like that on purpose, exactly so that people can't just grep for it to change it 😉. Motivation being that people should actually go through the code at least a bit and make appropriate modifications before deploying.
Closing this as a won't-fix issue. Feel free to open a new issue or PR if you have any other suggestions!
Issue Description
The code was not easily identifiable, making it more difficult to maintain or identify areas for modification.
E.g.
NimPlant/server/util/listener.py
Line 27 in ff66479
For some reason the Server header is encoded in hex. If it was in just plain text we could grep for the string and change it.
Improvement
Put the cleartext server header in the code instead of the hex representation because it's running on the server anyways.
Also, what was the reason to use base16 on it? It's unlikely anyone would run this on the target system with Python installed, so obfuscation isn't really necessary.
The text was updated successfully, but these errors were encountered: