LemmyNet/lemmy

[Bug]: NodeInfo implementation doesn't meet the 2.0 specification

MV-GH opened this issue · 0 comments

MV-GH commented

Requirements

  • Is this a bug report? For questions or discussions use https://lemmy.ml/c/lemmy_support
  • Did you check to see if this issue already exists?
  • Is this only a single bug? Do not put multiple bugs in one issue.
  • Do you agree to follow the rules in our Code of Conduct?
  • Is this a backend issue? Use the lemmy-ui repo for UI / frontend issues.

Summary

Lemmy's NodeInfo implementation doesn't meet the requirements of the NodeInfo 2.0 specification.

  • Specification doesn't allow any of the fields to be null.
  • Protocol field must contain at least one entry. (For private instances this is null)

See
http://nodeinfo.diaspora.software/docson/index.html#/ns/schema/2.0#$$expand

Steps to Reproduce

N/A

see current implementation https://github.com/LemmyNet/lemmy/blob/b152be79516adbd542d766df7b8a1928da590ced/crates/routes/src/nodeinfo.rs

Technical Details

This broke Sign in of private Lemmy instances in Jerboa see LemmyNet/jerboa#1487

Version

0.19.4

Lemmy Instance URL

No response