Skip to content
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

Other records accepted by Web NFC in smart poster #547

Closed
zolkis opened this issue Mar 6, 2020 · 5 comments
Closed

Other records accepted by Web NFC in smart poster #547

zolkis opened this issue Mar 6, 2020 · 5 comments

Comments

@zolkis
Copy link
Contributor

zolkis commented Mar 6, 2020

Captured from the discussion in #536 .
Questions:

  1. Should be other records allowed that are not explicitly specified in NFC Forum Smart Poster RTD?
    Yes, other records are allowed but applications MAY ignore it.
    Web NFC implementation is an application from NFC Forum point of view, so it can define its own policies on what to accept.

  2. What other records are allowed?
    Consensus:

  • MIME records are allowed
  • only one URI, type size and action record is allowed
  • the NFC Forum doesn't forbid other record types but there is a dispute whether to allow empty, absolute URL, external, local (other than Sp local), unknown records.
@zolkis
Copy link
Contributor Author

zolkis commented Mar 6, 2020

In my opinion,

  • Absolute URL records have a URL so must not be used in SP.
  • Empty records are anyway ignored on reads and seem to have no use case for writing.
  • Unknown records don't seem to have a use case at least for creation. Reads could support it.
  • External and local types should be OK to accept.

@kenchris
Copy link
Contributor

kenchris commented Mar 6, 2020

Just allow it all.. it might not make sense, but no reason to enforce arbitrary restrictions

@zolkis
Copy link
Contributor Author

zolkis commented Mar 6, 2020

OK - except absolute URL.
There MUST be one URL in the SP and that has to be a URI record.
Also, impls MUST remove URI/abs URI records from messages which contains a SP record.

zolkis added a commit to zolkis/web-nfc that referenced this issue Mar 6, 2020
Signed-off-by: Zoltan Kis <zoltan.kis@intel.com>
zolkis added a commit to zolkis/web-nfc that referenced this issue Mar 9, 2020
Signed-off-by: Zoltan Kis <zoltan.kis@intel.com>
@kenchris
Copy link
Contributor

Can this be closed @zolkis ?

@zolkis
Copy link
Contributor Author

zolkis commented May 20, 2020

Yes.

@zolkis zolkis closed this as completed May 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants