Closed Tjorriemorrie closed 7 years ago
Why is this hardcoded like that?
self.mongoclient = MongoClient('mongodb://guest:donald@52.201.173.151:27017/POKER')
I agree it shouldn't be. Any better suggestions?
On Sat, 11 Feb 2017 at 22:14, Tjorriemorrie notifications@github.com wrote:
Why is this hardcoded like that?
self.mongoclient = MongoClient('mongodb:// guest:donald@52.201.173.151:27017/POKER')
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dickreuter/Poker/issues/43#issuecomment-279179714, or mute the thread https://github.com/notifications/unsubscribe-auth/ABMrfr2L5yr8yyyqNylfND5iTNkKXtEwks5rbjLbgaJpZM4L-DGG .
Normally configuration generation. But I don't think that is suitable for this. I'll change it to env variables, then your defaults are preserved and allowing overwriting. If I come right I'll do a PR.
All that is needed is just a plain mongodb connection?
Yes. Ideally we would create a restful interface and then the public code will only connect to that but that's probably too much of a hassle.
With env variables you mean an external config file?
On Sat, 11 Feb 2017 at 23:10, Tjorriemorrie notifications@github.com wrote:
Normally configuration generation. But I don't think that is suitable for this. I'll change it to env variables, then your defaults are preserved and allowing overwriting. If I come right I'll do a PR.
All that is needed is just a plain mongodb connection?
— You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dickreuter/Poker/issues/43#issuecomment-279182661, or mute the thread https://github.com/notifications/unsubscribe-auth/ABMrfpZ4e_-aosXEa4CZkysAjfbysVlcks5rbj_cgaJpZM4L-DGG .
Is the mongodb server hosted on another server?
This doesn't make sense
def check_update(self, version):
cursor = self.mongodb.internal.find()
c = cursor.next()
current_version = c['current_version']
self.dl_link = c['dl']
latest_updates = c['latest_updates']
if current_version > version:
and cursor.next
gives StopIteration error because there's obviously no rows in mongo. How is this supposed to work?
No it's on the same server. Just a different collection. Do you get the same error when running the binaries?
On Mon, 13 Feb 2017 at 08:31, Tjorriemorrie notifications@github.com wrote:
Is the mongodb server hosted on another server?
This doesn't make sense
def check_update(self, version): cursor = self.mongodb.internal.find() c = cursor.next() current_version = c['current_version'] self.dl_link = c['dl'] latest_updates = c['latest_updates'] if current_version > version:
and cursor.next gives StopIteration error because there's obviously no rows in mongo. How is this supposed to work?
— You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/dickreuter/Poker/issues/43#issuecomment-279323088, or mute the thread https://github.com/notifications/unsubscribe-auth/ABMrfm46KE2gp7caOXoKEgV5y8Ni7hiDks5rcBT1gaJpZM4L-DGG .
Fixed the server
`
File "contextlib.py", line 59, in enter File "site-packages\pymongo\server.py", line 152, in get_socket File "contextlib.py", line 59, in enter File "site-packages\pymongo\pool.py", line 541, in get_socket File "site-packages\pymongo\pool.py", line 306, in check_auth File "site-packages\pymongo\auth.py", line 436, in authenticate File "site-packages\pymongo\auth.py", line 416, in _authenticate_default File "site-packages\pymongo\auth.py", line 188, in _authenticate_scram_sha1 File "site-packages\pymongo\pool.py", line 213, in command File "site-packages\pymongo\network.py", line 99, in command File "site-packages\pymongo\helpers.py", line 196, in _check_command_response pymongo.errors.OperationFailure: Authentication failed. Failed to execute script Pokerbot_downloader `