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

Add ongoing research #535

Closed
wants to merge 1 commit into from
Closed

Add ongoing research #535

wants to merge 1 commit into from

Conversation

kenchris
Copy link
Contributor

@kenchris kenchris commented Jan 13, 2020

RESEARCH.md Outdated
===

In order to make sure that the Web NFC API is future-proof, it is important to have a good overview of all the technologies that as part of NFC as well as those that support it. There are many technologies that are used by NFC but has broader usage and there are technologies not defined in the NFC Forum that people consider part of NFC.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

has --> have (broader usage).
Examples for the last clause?
Split up line?

RESEARCH.md Outdated
NFC Simple Exchange Protocol
---
The SNEP is a simple protocol for doing peer-to-peer (P2P) NDEF transfers. It is not supported by CoreNFC (iOS) and it is being deprecated on Android (where is branded Android Beam) due to security issues).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If need to quote the reason, provide link?

RESEARCH.md Outdated

### Conclusion

Due to SNEP being deprecated on Android and not supported by CoreNFC, Web NFC will not be adding P2P supporting over SNEP.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is CoreNFC? Link?

RESEARCH.md Outdated

Host Card Emulation
---
The idea of HCE was originally introduced on BlackBerry devices but today it is supported by Android out of the box.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Links would be welcome, if available.

RESEARCH.md Outdated
MiFare Classic and Plus cards can be configured to support NDEF, but are not NFC compatible, meaning they do require a NFC reader chip from NXP.

CoreNFC doesn’t support MiFare Classic (proprietary protocols) but people have had success reading NDEF formatted MiFare Classic cards.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Link to what is CoreNFC?
Maybe some more explanation would not harm, i.e. how MIFARE exposes NDEF, but it's up to you.

RESEARCH.md Outdated
---
NXP MiFare cards are quite popular and exists in different generations, with some NFC support, but also with support for proprietary protocols. As many phones have NFC readers from NXP, these additionally supports MiFare proprietary protocols. As iPhone’s NFC reader is also from NXP, CoreNFC supports MiFare protocols in addition to NFC protocols.

MiFare Classic and Plus cards can be configured to support NDEF, but are not NFC compatible, meaning they do require a NFC reader chip from NXP.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not NFC Forum compatible, I guess.

RESEARCH.md Outdated

### Conclusion

MiFare is proprietary technology and thus is not aligned with standards based approach of the web platform, thus proprietary MiFare protocols will not be supported, but as many tags and cards sold as NFC tags are actually MiFare Classic/Plus, we have decided that implementation may support Web NFC NDEF for MiFare Classic/Plus tags/cards.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split up line, replace second "thus" (or split sentence)?

RESEARCH.md Outdated

### Conclusion

T1T and T2T are usually used for NDEF and ISO-DEP is the most common low-level protocol for T4T so it makes sense to not support these protocols initially or at long as not supported by CoreNFC.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

T1T, T2T, T4T need links or definition.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Type 1 Tag :)

RESEARCH.md Outdated

Protocol support
---
Android supports most protocols expect SNEP which is deprecated.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

except

RESEARCH.md Outdated

If you are building an experience on top of NFC, then you know exactly what tech (tags, card) and protocols you support and getting notified for things you don’t support is not very useful.

A discoverability API additionally makes it easy to snoop info, and it makes permission handling potentially harder - like do you need to ask permission for everything when you start scanning, or when technology is detected?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Snoop info from where? Publicly readable tags?

RESEARCH.md Outdated
A discoverability API additionally makes it easy to snoop info, and it makes permission handling potentially harder - like do you need to ask permission for everything when you start scanning, or when technology is detected?

The FIFO order can be implemented on the web but it is quite hard and presumable hard for users tounderstand, especially as all async work in one reader session should block moving on to the next reader session in the FIFO queue:

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Split sentence.
tounderstand --> to understand

@zolkis
Copy link
Contributor

zolkis commented Jan 13, 2020

Nice work, needs more discussion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants