-
Notifications
You must be signed in to change notification settings - Fork 155
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 support for communication via NFC for Windows #481
Comments
Hi, sure, that'd be a nice addition. Could you share some code? |
I need to get approval from my managers to share code. |
JFI: I don't forget about "share some code". |
@martelletto does Yubikey have a Contributors License agreement (CLA) you can point me to? Before @kmenshikh can share his code our legal team wants to review Yubikey's CLA. thx |
@brett-berger, as long as the contributed code is under the same license as libfido2, we typically don't require a CLA: https://github.com/Yubico/libfido2/blob/master/LICENSE. Should that prove unsatisfactory to your legal team, could you please reach out to me at pedro [at] yubico [dot] com? Thank you. |
Hi, @martelletto I finally got approval. Some notices: (1) Provided code is based on the libfido2 v1.8.0 release. (2) libfido2 always use extended apdu format. (3) I didn't test scenario when both (WINHELLO and WIN_NFC) features are enabled. |
Thank you! For uniformity with the rest of the files in src/, would it be possible for the copyright notice to read: /*
* Copyright (c) 2022 Micro Focus PLC or one of its affiliates.
* Use of this source code is governed by a BSD-style license
* that can be found in the LICENSE file.
*/ or, if that's not acceptable, to include the entire license text in the copyright notice, so that it's all in one place? -p. |
@brett-berger could you answer @martelletto 's question? |
@martelletto this works for us. /*
|
adjust copyright as per feedback from Brett Berger in #481 (comment).
adjust copyright as per feedback from Brett Berger in #481 (comment).
adjust copyright as per feedback from Brett Berger in #481 (comment).
allows communication with PC/SC NFC readers on Windows, macOS, and Linux. disabled by default; enable with cmake -DUSE_PCSC=ON. based on code contribution from Micro Focus in issue #481; thanks! fixes
Inspired from: #249 (Add support for communication via NFC for MacOS #249)
Is it possible to do subj?
I've tried to do this (via Windows SmartCard API) and it works good on Windows 10 (on Windows 7 there are troubles).
"I've tried" means i extended functionality of libfido2.
It seems it's not hard enough.
The text was updated successfully, but these errors were encountered: