Jumoo / uSync.Complete.Issues

Public Issue tracker and roadmap for uSync.Complete
https://jumoo.co.uk/usync/complete/
2 stars 1 forks source link

Unrecognized escape sequence \T. #93

Closed Nuklon closed 3 years ago

Nuklon commented 3 years ago

Describe the bug

parsing "bin\Test" - Unrecognized escape sequence \T.
   at System.Text.RegularExpressions.RegexParser.ScanCharEscape()
   at System.Text.RegularExpressions.RegexParser.ScanBasicBackslash()
   at System.Text.RegularExpressions.RegexParser.ScanRegex()
   at System.Text.RegularExpressions.RegexParser.Parse(String re, RegexOptions op)
   at System.Text.RegularExpressions.Regex..ctor(String pattern, RegexOptions options, TimeSpan matchTimeout, Boolean useCache)
   at System.Text.RegularExpressions.Regex.IsMatch(String input, String pattern, RegexOptions options)
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source, Func`2 predicate)
   at System.Collections.Generic.List`1.RemoveAll(Predicate`1 match)
   at uSync.Expansions.Core.Services.SyncPackService.LoadRequiredFiles(SyncPackRequest request, String hashFile)
   at uSync.Expansions.Core.Services.SyncPackService.ImportFiles(SyncPackRequest request, String hashFile)
   at uSync.Publisher.Services.SyncPublishService.ImportWebsiteFiles(SyncPublishRequest request)
   at uSync.Publisher.Publishers.SyncRealtimePublisher.<PullImportFiles>d__55.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at uSync.Publisher.Controllers.uSyncPublisherApiController.<PerformAction>d__10.MoveNext()

To Reproduce Steps to reproduce the behavior: Import content from a remote site.

Additional context uSync 8.9.1.

KevinJump commented 3 years ago

Hi,

just to clarify is this via a 'sync-pack' using the exporter ?

Nuklon commented 3 years ago

I'm not that familiar with the names. What I did: select a node in Umbraco > ... > Do something else > Pull content... (push from other side has the same error) > keep everything selected. Then it loads for a bit (not that long) and this error comes up. If I uncheck most of the stuff below Advanced options it's going through.

KevinJump commented 3 years ago

Hi, i think you have an old/corrupt setting in your uSync.Publish.config file.

the system exclusions setting stops uSync coping over itself between sites. (and can be configured to not copy other things).

<systemExclusions>bin\\uSync,app_plugins\\uSync,bin\\App_Data</systemExclusions>

these are regex expressions so need the double slash. - i think you might have something in there that isn't a valid expression.

Kevin

KevinJump commented 3 years ago

Hi, just to update we might have an issue - checking it now,

we add and exclusion based on project name (based on any *.csproj file in the site root). its possible for us to then get an invalid regex expression loaded. so we might have to clean it up.

Nuklon commented 3 years ago

OK, thanks. I checked systemExclusions on both ends and it seems correct:

  <systemExclusions>bin\\uSync,app_plugins\\uSync</systemExclusions>

My csproj doesn't contain spaces (not sure if that's related though).

KevinJump commented 3 years ago

think i've fixed it but out of interest what is your csproj file name ?

Nuklon commented 3 years ago

It's TestUmbraco8.

rly-nozebra commented 3 years ago

I've got the exact same issue, except the file parsing is bin\MyProjectName

KevinJump commented 3 years ago

yeah - i think it's fixed - we just need to sanitize & text the next release to push it out.

rly-nozebra commented 3 years ago

It seems like it's the automated insert of the project dll in the exclude which misses a \

KevinJump commented 3 years ago

yeah it is. but also we've added a little bit of extra code to clean up any stuff that might get added via config and is incorrect.

rly-nozebra commented 3 years ago

@KevinJump Do you have any idea on, when 8.9.3 is released?

KevinJump commented 3 years ago

Hi - it should be today.

We where just halted as we where holding as someone said they had an issue, but i think it turns out they didn't so we should be able to do the checking and release stuff now.

rly-nozebra commented 3 years ago

Cool. Sounds great :-)

KevinJump commented 3 years ago

HI 8.9.3 is released now.