eileenmcnaughton / nz.co.fuzion.civixero

Synchronisation between CiviCRM & Xero
7 stars 14 forks source link

Fix handling of token renewal #126

Closed mattwire closed 1 year ago

mattwire commented 1 year ago

Ok, this fixes the access token issues in the latest version

eileenmcnaughton commented 1 year ago

Thanks @mattwire - hopefully one of the people who reported the issue can test it

eileenmcnaughton commented 1 year ago

/test

eileenmcnaughton commented 1 year ago

This behaves the same for me as without it - ie false negatives on the system check & works on the sync. I actually have connectors configured so that might be why I seem to have less problems. Anyway - given this doesn't break it for me & others have it actively broken & the test still runs I'm gonna merge this so that the others can better test

eileenmcnaughton commented 1 year ago

Hmm - I take it back - now the form is throwing the dreaded invalid_grant exception for me...

eileenmcnaughton commented 1 year ago

still I think it might be better to have this merged & then fix on top of it as there is some re-org in this

eileenmcnaughton commented 1 year ago

Dang - now my authorization is totally broken on the site I put this on

eileenmcnaughton commented 1 year ago

Phew got it working - it was the url - I just need to see if it needs to keep resetting now ....