По порядку:
1. try-except-pass - это нормальное использование обработки исключений, именно поэтому они исключения, что позволяют в случае любой ошибки сделать что-то одно, без необходимости перечислять каждую возможность, без необходимости создавать перечисления всех возможных ошибок и типы-суммы для вызываемых внутри функции других функций с ошибками. Можно просто любой код обернуть в try-except и поймать таким образом любые исключения, даже не зная о том, могут ли они случиться.
2. eval в питоне называют нехорошим из-за возможной подстановки кода. А выполнение интерпретатором кода не считается? Я ведь могу любую программу на питоне взять и подменить в ней исходники, и они выполнятся. В общем eval\exec и прочее - это выполнение произвольного кода. Любой запуск программы - это выполнение произвольного кода, поэтому глупо запрещать выполнение произвольного кода в выполняемом произвольном коде.
3. Использование ключей в коде - это самый простой способ работать с штуками, которые требуют ключи. Главное - не допустить попадание ключа в публичный доступ (менять при публикации кода).
4. Использование небезопасных шифров и протоколов... Ну тут что сказать? Оно устаревает и признается небезопасным быстрее, чем код писать успеваешь. Переключишься с md5 на sha2 - а через день и sha2 обнебезопасится. Глупо называть из-за таких вещей код небезопасным.