search

Podcastindex-org / podping-hivewriter

The hive writer component of podping.
MIT License
15 stars 5 forks source link

Feature/1.1/#27 add medium reason to cli interface #34

Closed agates closed 2 years ago

agates commented 2 years ago

Closes #27

brianoflondon commented 2 years ago

Added to the CLI help otherwise all looks good.

brianoflondon commented 2 years ago

I think I've reviewed everything now.

brianoflondon commented 2 years ago

I'm trying to figure out why tests are suddenly failing.

agates commented 2 years ago

Is it just a timeout thing? I'll look into it too

brianoflondon commented 2 years ago

It's not just timeout. Fix coming soon.

when trx_id return from the post to Hive was removed it still remained in 

    async def _iri_batch_handler_loop(self):

but within a try: block so that loop failed silently and never ran again.

brianoflondon commented 2 years ago

And by the way the only way I could debug and find this was by my old fashioned method of running the server in debug mode and having another hive-watcher running and sending URLs in via the ZMQ queue. I could see the batch handler run once then crap out.