lispercat / sailpoint-iiq-dev-accelerator

VSCode extension to accelerate and facilitate development by directly interacting with the target Sailpoint IIQ server
MIT License
31 stars 10 forks source link

Escape character in tokenization #8

Closed stsang closed 3 years ago

stsang commented 3 years ago

Could you take this into consideration? When tokenizing variables, the \ char is not recognized as an escape character, e.g. when there are two \ it should be replaced to a single \

%%FTP_PATH%%=C:\\Temp\\Files

This is causing backwards compatibility with the Eclipse plugin.

lispercat commented 3 years ago

Let me confirm this: In your config file the line actually goes: %%FTP_PATH%%=C:\Temp\Files Correct?

On Tue, Nov 3, 2020 at 9:34 AM stsang notifications@github.com wrote:

Could you take this into consideration? When tokenizing variables, the \ char is not recognized as an escape character, e.g. when there are two \ it should be replaced to a single \

%%FTP_PATH%%=C:\Temp\Files

This is causing backwards compatibility with the Eclipse plugin.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/lispercat/sailpoint-iiq-dev-accelerator/issues/8, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACCZB2PM7BZLG4D7LYSYUTSOAIIDANCNFSM4TI2JDVA .

stsang commented 3 years ago

Yes, that's what I meant. For some reason github escaped the double char in my post.

lispercat commented 3 years ago

@stsang I made a fix in 1.0.19. Please check

stsang commented 3 years ago

Verified, thanks!