💾 Archived View for bbs.geminispace.org › u › gemalaya › 11246 captured on 2024-08-18 at 20:46:10. Gemini links have been rewritten to link to archived content
⬅️ Previous capture (2024-07-09)
-=-=-=-=-=-=-
Re: "SSL Errors from misfin listener (cipres a45df40)"
@skyjake Cheers ! Reading the code now. The "model" module is ... pretty large ^_^ Could be good to turn it into a package ?
2023-10-28 · 10 months ago
🕹️ skyjake [...] · 2023-10-28 at 19:50:
@gemalaya (Sorry for derailing the thread...) Personally I have nothing against large modules. There should be some functional justification why it would need to be a package. Currently the module is mainly one hefty class that talks to the SQL database.
That being said, I'm sure the whole code base could be organized in a neater way. However, I'm more of a pragmatist and tend to prioritize cleanup refactorings pretty low if it's just for the sake of tidiness.
😺 gemalaya · 2023-10-28 at 20:50:
@skyjake I agree, it's just that there are a dozen classes, with a class per entity, and then there's the big "Database" class, where there's a lot of logic. This lead me to think that putting the entity classes (Post, User, Subspace, etc ..) in a separate module and have the Database object in another module would make it easier to read and maintain, but it's not a sufficient reason. Great work, i like how it depends on so few libraries.
SSL Errors from misfin listener (cipres a45df40) — I'm using the cipres misfin client (commit a45df40 w/ local python 3.7 compatibility patches) and recently started seeing SSL errors after a few weeks of sucessful use. Error [('SSL routines', '', 'http request')] Aborting due to exception. Error [('SSL routines', '', 'wrong version number')] Aborting due to exception. Error [('SSL routines', '', 'wrong version number')] Aborting due to exception. Error [('SSL routines', '', 'wrong version...