Check and confirm behavior on various URLS

abhidas17695 / dweb-ext

0 stars 0 forks source link

Closed mitra42 closed 6 years ago

mitra42 commented 6 years ago

Please test each of these, and create issues if any don't work as expected (note the problem could be in the bundled code of course) .

dweb.archive.org -> should end up booting into the archive site
dweb.foo.com -> Should end up failing and trying the real dweb.foo.com (it probably doesn't yet)
www.iskme.org -> should never be intercepted and should go direct to www.iskme.org
dweb.me -> Should not be intercepted - it could end up in a loop - but need to figure out how

abhidas17695 commented 6 years ago

Code addresses this issue although present version of extension assumes all URLs to be dweb.archive.org

mitra42 commented 6 years ago

Release in #22 has a problem with the dweb.foo.com example

mitra42 commented 6 years ago

Add to the list of test URLS

https://dweb.archive.org/details/commute -> should go to https://dweb.me/archive/archive.html?item=commute (which is what domain resolution should be sending back)
https://dweb.archive.org?verbose=true -> should go to the site with verbose=true (or any other parameters) set.

mitra42 commented 6 years ago

These are all working. Leaving this open just for testing purposes in case we make changes.

mitra42 commented 6 years ago

For now, please retest each version against the URLs above, ( I notice you just caught a bug with wrongly catching other URLs)

mitra42 commented 6 years ago

WE can close this as we aren't editing that code any more.