-
Notifications
You must be signed in to change notification settings - Fork 943
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 ECC profiles #2398
Open
mrsuciu
wants to merge
99
commits into
OPCFoundation:master
Choose a base branch
from
mrsuciu:ecc_support
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 43 commits
Commits
Show all changes
99 commits
Select commit
Hold shift + click to select a range
294192f
Squash proto ecc (#1998)
mregen fd3fcd1
clean a few warning
mregen dee6088
Instantiate HMAC instance depending on provided algorithm.
mrsuciu b17271c
fix build
mregen d63925f
Added ApplicationCertificates XML tag (bcw compat)
mrsuciu 5b1fe70
Use ListOfCertificateIdentifier for configuration
mrsuciu ab6d8fb
Use ApplicationCertificates tag
mrsuciu d23450d
Added new Interface AddSecurityConfiguration(
mrsuciu 8fc81d3
Reenabled special cases in ValidateCertificateType
mrsuciu b1cd86e
Use KeySize property value if set under IsECSecureForProfile method
mrsuciu 108f7ed
Check minimKeySize for ApplicationCertificates
mrsuciu 9a25264
Keep and mark as obsolete ApplicationInstance methods for bacword compat
mrsuciu a2e0a4b
Fix CfgManager UpdateCertificate()
mrsuciu a0e4b0f
Added missing interface implementation
mrsuciu 97b4c87
Added CertificateTypeString to improve visual appearance
mrsuciu 9efe4a7
Commented ECCUtils code
mrsuciu a01d276
Corrected ConsoleRefClient Configuration
mrsuciu f165dee
net48 has null Oid values for Brainpool curves
mrsuciu 0b6d74f
Merge branch 'workeccV1' of https://github.com/mrsuciu/UA-.NETStandar…
mrsuciu 58f583b
Updated tests to new API
mrsuciu 453ad95
Default certificate type for "old" style configuration is RsaSha256Ap…
mrsuciu 11f3085
SemaphoreSlim is not reentrant => removed from ResetValidatedCertific…
mrsuciu 48cce5c
Fixed loading of disposed cached certificates
mrsuciu 1ada020
Modified projects to correctly use ECC_SUPPORT flag
mrsuciu c7ebfe1
Handle EphemerousKey from Server side
mrsuciu e62ad97
Added minimal test
mrsuciu 1a8ab07
Added UserIdentityToken Encypt/Decrypt functionality
mrsuciu 0ed43b9
Preserve key material in UserIdentityToken encryption/decription
mrsuciu f41f10a
Removed ECC support from NETSTANDARD2_0 (introduced bu UserIdentityEn…
mrsuciu c6a10d7
Added UserIdentity encryption positive roundtrip unit tests
mrsuciu 57a79fb
Propagated clientIssuerCertificates to UserIdentityToken encryption
mrsuciu f37a392
Save the userTokenSecurityPolicyUri per Session (needed in reconnect)
mrsuciu 2c4d446
Added FindUserTokenPolicy methods which support providing tokenSecuri…
mrsuciu 7f00e7a
Added eccServerEphemeralKeu to saved session secrets
mrsuciu fd2bae7
Merged conflicts and addapted code
mrsuciu 89be8d6
Addapt code to merge changes
mrsuciu 8beb595
Added ReentrantSlimSemaphore
mrsuciu 1e579b1
Added EccProfiles.md
mrsuciu 0146322
merge master
mregen d1aa4e9
improve project setup for ECC
mregen 0fc78b0
Merge with master
mrsuciu f009283
Merge remote-tracking branch 'origin/master' into ecc_support
mregen c500a6d
merge master
mregen 46e17f8
merge master
mregen 870dbc7
fix build
mregen 145bfdd
Merge remote-tracking branch 'origin/master' into ecc_support
mregen 1447154
Removed unused code and corrected some
mrsuciu 5506269
ECC cert fixes
mregen d737c52
fix when app cert can not be loaded due to lack of trust or CRL
mregen 0d03fa0
fix a merge conflict (includes releaxed validator for cert loading)
mregen ba26d64
fix semaphore hang
mregen 3306aa9
fix CertificateTypes Provider & CertificateFactory
romanett 84873bf
client session assync initialize
mrsuciu 5fdc79d
Deprecated Utils.Nonce
mrsuciu 058cb96
Simplify Nonce.CreateNonce method, added NonceTests UnitTest
mrsuciu 55c97b0
Removed #if ECC_SUPPORT conditional compilation statements related to…
mrsuciu 5760ac7
Move async code out of the constructor
mrsuciu a3d1469
Implemented SetECDsaPublicKey, ReentrantSemaphoreSlim removed
mrsuciu 7bcce20
Merge with latest master
mrsuciu c53858e
Added MaxChannelCount of 100 to ServerFixture
mrsuciu 303c7b1
Set version 1.5.375-ECC-preview
mrsuciu 2f4aef0
Removed ECDsaCng validation for ECDsa (OS platform dependent)
mrsuciu cff0414
fix CreateNonce function for invalid input data
romanett c1a9fd2
ignore test on OSX
romanett d54fbc4
Remove NoWarn tag
mrsuciu 58f1ff7
Ignore GetEndpoints call exceptions on platforms other than Windows f…
mrsuciu 8fdaf6d
Use RSA minimum certificate size of 2048 (OpenSSL on Linux does not a…
mrsuciu e165873
Increase test timeout to 45 minutes
mrsuciu 46839f8
merge with errors
mrsuciu b6ce654
Fix build errors
mrsuciu b5db0fb
Fix validation of Nonce length
mrsuciu 4bc277f
add ECC polices to Client Security Level calculation
romanett b7f3ff8
Remove wrong propagation of minKeySize into RSA application certifica…
mrsuciu 977680d
Fix behaviour of flag AddAppCertToTrustedStore
mrsuciu e973433
Merge with lastest Master
mrsuciu a28d901
Fix netstandard2.0 compilation
mrsuciu 505d325
Changed versionHeightOffset to 70
mrsuciu 8d28310
Modirfied Opc.Ua.Client.cproj to generate the APICompat suppression file
mrsuciu 4280d0c
merge main into ecc_support
mregen 8b998ad
do not duplicate the code for additional header user ecc secret
mregen 3fcb5e4
fix formatting
mregen f20f206
simplify code
mregen 9916621
Merge remote-tracking branch 'origin/master' into ecc_support
mregen 32c2f9e
add test and a few missing bits in sessionconfiguration
mregen 15574d4
Sequence Header sequenceNumber calculation depends on Security Policy
mrsuciu 37c44f7
Removed Libraries/Opc.Ua.Configuration/Opc.Ua.Configuration.csproj.bak
mrsuciu 9b4ecf6
improve a comment
mregen f5992a2
merge main
mregen 1a47848
add a review comment
mregen 3e6329e
Merge remote-tracking branch 'origin/master' into ecc_support
mregen 88e1ac6
fix a comment
mregen c6b0cc3
Merge remote-tracking branch 'origin/master' into ecc_support
mregen 7d1496d
Merge with latest master
mrsuciu f6f6ca3
Unified SessionConfiguration
mrsuciu d79719f
Unified UserIdentityToken Encrypt/Decrypt
mrsuciu 7d1da41
Removed MinimumECCertificateKeySize
mrsuciu d336990
Merge with latest master
mrsuciu 91f6852
Initial sequenceNumber value depends on policy;
mrsuciu 69ab50a
Simplify GetNewSequenceNumber
mrsuciu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -1054,4 +1054,4 @@ | |
<PreBuildEvent> | ||
</PreBuildEvent> | ||
</PropertyGroup> | ||
</Project> | ||
</Project> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -170,4 +170,4 @@ | |
<PostBuildEvent> | ||
</PostBuildEvent> | ||
</PropertyGroup> | ||
</Project> | ||
</Project> |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
isn`t this obosolete?