Skip to content
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

Core #7

Closed
wants to merge 3 commits into from
Closed

Core #7

wants to merge 3 commits into from

Conversation

stumpyfr
Copy link

@stumpyfr stumpyfr commented Feb 2, 2017

To follow up #6 , here a first version compatible .NetCore.
Based on Netstandard1.6

Both version should work at the same time, I keep the "old" project Udger.Parser in the solution for now but I disabled it.
We will probably be able to remove it very soon.

@lloydpowell88
Copy link

lloydpowell88 commented Jun 23, 2017

The core project doesn't detect any of the ua properties, they work in the "old" project but not in the new.

I guess there's an issue matching on the clientRegstringList.

For reference the user agent string is
Mozilla/5.0 (Windows NT 6.3; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3029.110 Safari/537.36

@stumpyfr
Copy link
Author

stumpyfr commented Jun 24, 2017 via email

@lloydpowell88
Copy link

lloydpowell88 commented Jul 19, 2017

I did have a look but couldn't figure out the area causing issues unfortunately. If you could point me in the correct direction then that would be greatly appreciated :-)

@lloydpowell88
Copy link

Hi @stumpyfr, sorry to be a pain but by any chance did you get chance to take a quick look at the ua properties? I'm hoping it's something really simple that I just can't spot! Thanks :-)

@stumpyfr
Copy link
Author

Sorry, I really dont have time to check that, we dont use it anymore ourself.
Probably not a bad idea to start over, it's looks like the origin has some changes since I started.

@stumpyfr stumpyfr closed this Jul 31, 2017
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