-
Notifications
You must be signed in to change notification settings - Fork 76
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Outdated example data for breaking change #58
Comments
Hey @d-Rickyy-b, apologies for the oversight! I'll work to get them changed next time I'm working on this.
Yeah actually looking at it it seems as though my github tags have been lost, but we did have things versioned well IIRC :(. I guess my recent history rewrite to reduce the stored files size wiped tags as part of it. At that point the idea is mostly just sticking to a version you're aware of instead of just running master constantly. As an alternative if you want to grab specific version commits (I'll go back and re-tag when I go fix the other stuff) you can use https://github.com/CaliDog/certstream-server/blame/master/mix.exs and click the "go back in history" icon to find the version you were using previously. Sorry for the breakage 😅, if you have other ideas about how to manage versions for a project like this other than git tags/releases I'm happy to consider them! |
Ugh yeah all my tags are gone :(. The metadata seems to still be there in Github so I wonder if I restore the tags it'll restore my releases as well? 🤔 https://github.com/CaliDog/certstream-server/releases/tag/1.4.0 |
Sorry to hear that. But great that you were able to narrow down the issue quite fast.
I was rather talking about your hosted version of certstream on certstream.calidog.io. Maybe host multiple websocket listeners on different endpoints - such as Little bit of OT: Thanks for building this tool btw. I am currently in the making of (yet another) certificate monitoring tool based on top of certstream. |
This was removed upstream. See: * CaliDog/certstream-server#58 * CaliDog/certstream-server#56
In the commit 96fcfc0 / PR #56 you removed the 'chain' and the 'as_der' fields from the data. This is quite a breaking change that also hasn't been updated in the README file yet!
certstream-server/README.md
Lines 118 to 161 in 26dff90
Nor have the "subprojects" such as the python module gotten an updated README:
https://github.com/CaliDog/certstream-python#example-data-structure
Also the certstream website does not show this fact immediately: https://certstream.calidog.io/ (under "Certificate Update").
Would you mind fix these examples? Especially important for newcomers!
PS: I noticed this only because my application suddenly stored NULL values for the "as_der" and some chain data in the database. As kind of suggestion: It would be lovely to have some sort of API versioning so that applications don't just break from one day to another because API behaviour changed.
The text was updated successfully, but these errors were encountered: