-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat: add sas 9 remote (via IOM bridge) support #592
Merged
Merged
Changes from 49 commits
Commits
Show all changes
53 commits
Select commit
Hold shift + click to select a range
dc7b03f
get iom bridge working
998baf5
add basic profile setup
3286b16
add password storage
cfdf4a0
clean up code/fix password prompt
e4afd38
self code review
e5a6f0e
fix tests
303548b
move com->itc
8347560
add test for fetch file
217da85
fix tests for windows
d4eadde
update CHANGELOG / connect-and-run
0f445a0
Update README
046459a
Run npm run format
0b5919f
Rename COMSession -> ITCSession
c3474bf
Update const/comment/default port
7cf2255
Update connect-and-run
16a0309
Use components/ExtensionContext
591357b
store password when fetched from session
4fd49ff
introduce namespaced secret storage
5efd0ec
update how sessions are stored/results are fetched
5ec2211
wip - implement cancel
aff5798
clear password on session close
cf93f62
implement cancel
4473702
Update documentation/npm run format
a091f7c
cleanup testing method
c740bd6
update translations
0ac2c03
move includes -> endswith
44cec6a
fix filepath
010a799
run npm run format
138ad73
fix: set correct encoding on powershell client
smorrisj 0a00464
chore: fix tests
smorrisj 3830454
chore: run test with en-US locale for deterministic behavior
smorrisj 72887db
fix cancelling
895d7e8
fix lint/format issues
85a267b
Merge branch 'main' into feat/sas9itc
scottdover 3dee692
Update cancel process
bf8ac6a
Update cancel process
c644317
Merge branch 'main' into feat/sas9itc
scottdover d1c2520
Merge branch 'main' into feat/sas9itc
scottdover 1ffda86
resolve sas 9 popup error
4663aec
Merge branch 'feat/sas9itc' of github.com:sassoftware/vscode-sas-exte…
e5eeff5
fix test
83eae49
Merge branch 'main' into feat/sas9itc
scottdover 0d7667a
Merge branch 'feat/sas9itc' of https://github.com/sassoftware/vscode-…
1dfdea8
add chcp 65001 to powershell command
668b51f
Merge branch 'main' into feat/sas9itc
scottdover 4c7b373
upgrade prettier, run npm run format, fix tests
3b80433
fix running w/o ods results
5c554aa
add itc connection test script
04c5829
fix ods results html
ad48cd1
collect all bytes before string conversion
a589058
Revert "collect all bytes before string conversion"
88c6b54
use filestream over streamwriter
538597b
update chunk size
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
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
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.
This is probably ok for now, but at some point in the future, we should probably use the ODS IOM events instead (namely ODSComplete), similar to the way EG and SAS Studio handle. Example C# code:
ODS_1_1 _odsService = (ODS_1_1)ws.ODS;
CIODSFileEvents_1_1_Event _odsFileEventsFor93 = ws.ODS as CIODSFileEvents_1_1_Event;
_odsService.FileOpen += HandleOdsFileOpen;
_odsService.FileClose += HandleOdsFileClose;
_odsService.DirectoryBegin += HandleOdsDirectoryBegin;
_odsService.AnchorElement += HandleOdsAnchorElement;
_odsFileEventsFor93.ODSComplete += HandleOdsComplete;
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.
Hey @clangsmith . Thanks for this. I linked this comment to the IOM library panel issue and will take a look when I'm working on that.