omada-api icon indicating copy to clipboard operation
omada-api copied to clipboard

Add functionality to allow for uploading SSL certificates to Controller

Open jsestrich opened this issue 2 years ago • 4 comments

I was looking for a way to automate uploading new SSL certificates to my controller and found your API. I've been successfully using this code to update my SSL certificates on my controller, so I figured I'd contribute back what I added.

jsestrich avatar Mar 17 '23 05:03 jsestrich

Looks like @ghaberek isn´t maintaining this repo anymore. Maybe you want to contribute to my fork.

thannerfabian avatar May 05 '23 14:05 thannerfabian

Looks like @ghaberek isn´t maintaining this repo anymore.

Please refrain from making assumptions about my involvement in my own project. I "maintain" this when I can, as best I can. I'll occasionally sweep through and merge some PRs and fix up some things but I've no specific intent or direction for anything. If anyone wants to provide more regular maintenance and improvement I'll gladly add them as a collaborator so they can push without waiting for me to approve a PR.

ghaberek avatar May 05 '23 17:05 ghaberek

It looks like your "tabify" commit also includes most of your code changes. I'm not sure how you would go about it, but I'd prefer to merge only the code changes and we can figure out "tabs vs spaces" changes later (FYI PEP-8 says to prefer spaces).

ghaberek avatar Jun 23 '23 15:06 ghaberek

phew. Took me a minute to get my git-fu back.

I realized I shouldn't commit all the individual parts here (it's just cluttering the log), and squashed it all into a single commit. I think this should be "just the code changes" now.

The tabify thing is a bit of a red-herring. If you look at the chain closely, I accidentally converted everything to spaces earlier in the chain and then later converted it back to leave things the way I found them.

I don't have a strong opinion on tabs vs. spaces (I'm happy to follow suit with whatever you'd like to do), but if you'd like I can also update everything to spaces in a separate pull request.

jsestrich avatar Jul 08 '23 05:07 jsestrich

Just wanted to bump this thread. I think this code is ready for your review.

I have been using this successfully for a few months for auto-renewing my SSL certs.

jsestrich avatar Apr 30 '24 04:04 jsestrich