Skip to content

Conversation

@danyarmoski-medsol
Copy link

There is an issue with the current script where as of 3 days ago the code changed and there is Regex that is converting the value of Boolean to start with a capital letter. For example DevOpsFileEvent_properties_typeProperties_ignoreEmptyBlobs it was converting that to value True with a capital T which results in error ##[error] Conversion from JSON failed with error: Unexpected character encountered while parsing value: T. Path 'properties.typeProperties.ignoreEmptyBlobs', line 21, position 26. from Line: 680 ##[error]Conversion from JSON failed with error: Unexpected character encountered while parsing value: T. Path 'properties.typeProperties.ignoreEmptyBlobs', line 21, position 26. To fix this you add the .ToString().ToLower() to the end of line 475. Which will not convert boolean values to capital letters and makes the error go away.

There is an issue with the current script where as of 3 days ago the code changed and there is Regex that is converting the value of Boolean to start with a capital letter.  For example DevOpsFileEvent_properties_typeProperties_ignoreEmptyBlobs it was converting that to value True with a capital T which results in error ##[error] Conversion from JSON failed with error: Unexpected character encountered while parsing value: T. Path 'properties.typeProperties.ignoreEmptyBlobs', line 21, position 26. from Line: 680
##[error]Conversion from JSON failed with error: Unexpected character encountered while parsing value: T. Path 'properties.typeProperties.ignoreEmptyBlobs', line 21, position 26.
To fix this you add the .ToString().ToLower() to the end of line 475.  Which will not convert boolean values to capital letters and makes the error go away.
@danyarmoski-medsol
Copy link
Author

#510

@chez-charlie
Copy link

#sign-off

@danyarmoski-medsol
Copy link
Author

#sign-off

I am pretty sure that this is already addressed in the latest release of the script and nuget package for data factory automation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants