-
Notifications
You must be signed in to change notification settings - Fork 64
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
Option of using OpenSSL for authentication between RTI and federates #1432
Merged
Changes from all commits
Commits
Show all changes
62 commits
Select commit
Hold shift + click to select a range
32eb917
Added HMAC authentication between RTI and federates, using federation…
Jakio815 68085e9
Target Config for auth is set false. auth option is useable for targe…
Jakio815 750ea8b
Added SimpleFederatedAuth.lf for test of auth option
Jakio815 573ad15
Added generating -a option to RTI
Jakio815 e46d858
Added FEDERATED_AUTH when targetConfig.auth is true
Jakio815 cb958c0
updated submodule
Jakio815 2d3224b
Merge branch 'master' into auth
Jakio815 bd058c5
Fixed comments for target property AUTH
Jakio815 152b9d2
Make definition for federate authentication more readable.
hokeun feaa943
Rename the authentication test to make it more descriptive. Add DEBUG…
hokeun d9d1aec
Update reactor-c submodule.
hokeun 8433132
Add comments and DEBUG logging to SimpleFederatedAuthenticated.lf.
hokeun 8bb1842
Merge branch 'master' of github.com:lf-lang/lingua-franca into auth
hokeun fce6884
Added comments
Jakio815 f38f04b
Merge branch 'master' of github.com:lf-lang/lingua-franca into auth
Jakio815 f99e61c
Add fixmes to update auth option type from boolean to string.
Jakio815 678fccb
Merge branch 'master' of github.com:lf-lang/lingua-franca into auth
Jakio815 3907b22
Merge branch 'master' of github.com:lf-lang/lingua-franca into auth
Jakio815 494812c
Add PRIVATE keyword to openssl code generator.
c7511b9
Updated reactor-c version
Jakio815 5363089
Add -DAUTH=ON for tests
8c1f403
Update reactor-c version
d50dc23
Update reactor-c version
308554e
Add mac os dependency openssl
eb5da4b
Formatting SimpleFederatedAuthenticated.lf
7177e4e
Added cmake mac options
ae37da5
Minor revisions
c95a258
Test CI Tests
d889a47
Add openssl mac links
fefcc12
changed test branch temporarily
085dd9e
Formatted SimpleFederatedAuthenticated.lf
04ee608
Merge branch 'master' of github.com:lf-lang/lingua-franca into auth
cf12709
add path to mac dependencies
2741cfd
Addmore to mac dependencies
0238862
Added mac os options in cmake generator
f35c0c3
Test Mac environment
a998c89
Fix bugs
0568180
Buf fix
4e7c972
Add exports
26ac623
Added CCPP target properties
b5f7602
Test shorter tests
dbb2970
Bug fix
ef7f66f
Bug fix
c8674b1
Test added
81edc54
FIx bugs
8248a60
FIX bugs
a9b91c9
Change orders of finding openssl
8f452c2
Changed to SET path
2160e58
Remove unneeded lines
2b272ac
Restore ci tests
b41dbca
Restore ci tests
9251b9c
Update reactor-c
f12b27f
Test export
00d5164
Fix bugs
88e1a46
FIX bugs
7082ffc
FIX bugs
d3565d3
Testing variables
a82a76f
Restore git CI tests
896ad59
Fix bugs
49a7aaf
Merge branch 'master' into auth
Jakio815 7ffc785
Removed Fixmes
524e556
Merge branch 'master' of github.com:lf-lang/lingua-franca into auth
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
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
Submodule reactor-c
updated
10 files
+4 −4 | .github/workflows/ci.yml | |
+20 −0 | core/federated/RTI/CMakeLists.txt | |
+11 −0 | core/federated/RTI/README.md | |
+84 −2 | core/federated/RTI/rti.c | |
+5 −0 | core/federated/RTI/rti.h | |
+90 −0 | core/federated/federate.c | |
+90 −78 | core/reactor_common.c | |
+50 −0 | include/core/federated/net_common.h | |
+1 −1 | include/core/lf_types.h | |
+1 −1 | lingua-franca-ref.txt |
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 |
---|---|---|
|
@@ -58,7 +58,13 @@ | |
* @author{Marten Lohstroh <[email protected]>} | ||
*/ | ||
public enum TargetProperty { | ||
|
||
/** | ||
* Directive to allow including OpenSSL libraries and process HMAC authentication. | ||
*/ | ||
AUTH("auth", PrimitiveType.BOOLEAN, | ||
Arrays.asList(Target.C, Target.CCPP), (config, value, err) -> { | ||
config.auth = ASTUtils.toBoolean(value); | ||
}), | ||
|
||
/** | ||
* Directive to let the generator use the custom build command. | ||
|
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 |
---|---|---|
@@ -0,0 +1,23 @@ | ||
/** | ||
* This simple test checks if federate authentication works by adding `auth` | ||
* target property. | ||
*/ | ||
target C { | ||
timeout: 2 secs, | ||
build-type: RelWithDebInfo, | ||
auth: true, | ||
logging: DEBUG | ||
} | ||
|
||
reactor Fed { | ||
input in: int | ||
output out: int | ||
} | ||
|
||
federated reactor { | ||
fed1 = new Fed() | ||
fed2 = new Fed() | ||
|
||
fed1.out -> fed2.in | ||
fed2.out -> fed1.in | ||
} |
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.
Only adding export OPENSSL_ROOT_DIR="/usr/local/opt/openssl" does not work, because it seems that every run opens a new bash.