Update nodeinfo #446
No reviewers
Labels
No labels
A: API
A: Backend
A: Federation
A: Front-End
A: I18N
A: Meta
A: Security
Build
C: Bug
C: Discussion
C: Enhancement
C: Feature
Compatibility
Dependency
Design
Documentation
Good first issue
Help welcome
Mobile
Rendering
S: Blocked
S: Duplicate
S: Incomplete
S: Instance specific
S: Invalid
S: Needs Voting/Discussion
S: Ready for review
Suggestion
S: Voted on Loomio
S: Wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: Plume/Plume#446
Loading…
Add table
Reference in a new issue
No description provided.
Delete branch "update_nodeinfo_433"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Fix #433
I added the repo link to Cargo.toml so that
software.repository
could be configurable like @rhaamo suggested. I don't know if it's ok to includesoftware.repository
without bumping the schema version, but I didn't know if that would break any clients that parse nodeinfo with a hardcoded schema version.software.repository
is nodeinfo 2.1 only, you can however have 2.0 and 2.1 side by side, only the json for 2.1 will returns the repository key.But that will introduce another thing to do, plume use
https://xxx/nodeinfo
so it would need to becamehttps://xxx/nodeinfo/version
or something like that.keeping 2.0 would be good anyway for compatibility.
and for the schema version, if you add repository in a 2.0 version it will break on any parser forcing a validation, since 2.0 schema doesn't allow repository.
Ok. I'll update that. Is it ok to make it two separate endpoints,
/nodeinfo/2.0
and/nodeinfo/2.1
since those are the only ones we provide or should the version be parameterized?Codecov Report
Thank you! For the versions, adding a parameter in the route could indeed do the job. And I'm not sure the
json!
macro allows it, but if possible only add the repository field if the requested version is 2.1Thank you! 👍