doenietzomoeilijk / navbot

Automatically exported from code.google.com/p/navbot
0 stars 0 forks source link

Not compatible with new browser features/version in Dominion. #61

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Status bar indicates the page has loaded but nothing is displayed. 
Selecting "View Page Source" from right-click menu displays blank html file.

Original issue reported on code.google.com by PaulPete...@gmail.com on 2 Dec 2009 at 4:55

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Same thing here.
Shouldn't this issue have higher priority? It makes NavBot completely unusable.

Original comment by WolfPlag...@gmail.com on 2 Dec 2009 at 3:03

GoogleCodeExporter commented 9 years ago
I am having the same problem... NAVBOT worked fine until yesterday when they 
applied 
the patch. Now all I see in the game browser is a white page after loading..

Original comment by Anders.F...@gmail.com on 2 Dec 2009 at 6:32

GoogleCodeExporter commented 9 years ago
Same issue.  Higher priority is agreed.

Original comment by bjwim...@gmail.com on 3 Dec 2009 at 5:59

GoogleCodeExporter commented 9 years ago
So what do we use until they fix NavBot?
It's really sad that Dominion killed it. Please fix it.

Original comment by miske...@gmail.com on 5 Dec 2009 at 12:07

GoogleCodeExporter commented 9 years ago
It is sad, but what's strange is there has been no comment or post from the
developers acknowledging this issue.  You'd think it would be a top priority to 
add
under "known issues"????  

Original comment by PaulPete...@gmail.com on 5 Dec 2009 at 3:14

GoogleCodeExporter commented 9 years ago
I'm a developer too. I just downloaded NavBot, thinking maybe the new browser 
doesn't
detect ports other than normal (http port 80). I recompiled NavBot to just run 
on
localhost:80 , but it sadly didn't work... 

Original comment by miske...@gmail.com on 6 Dec 2009 at 1:58

GoogleCodeExporter commented 9 years ago
It also manages to connect to localhost:80 if i run apache http server.
So, maybe they made the new browser detect if it's a NavBot generated page. But 
I
wouldn't like to speculate and make people give up on NavBot, so maybe there is
something that needs to be fixed. Need to ask on eve forums.

Original comment by miske...@gmail.com on 6 Dec 2009 at 2:06

GoogleCodeExporter commented 9 years ago
It's actually much more likely that the new browser simply cannot read the 
headers
that Navbot is using. Worst case scenario, we could make navbot work outside of 
eve,
in a normal window... 

Original comment by miske...@gmail.com on 6 Dec 2009 at 2:24

GoogleCodeExporter commented 9 years ago
I am fine with an out of browser type program...  I love this mainly because it 
uses
my own export that no one else shares and can manipulate.  So as long as it 
keeps
that feature I am happy to alt + tab or just use another monitor.

Original comment by bjwim...@gmail.com on 6 Dec 2009 at 5:14

GoogleCodeExporter commented 9 years ago
Has anyone figured out a work around to this?

Original comment by adam.h.y...@gmail.com on 12 Dec 2009 at 8:37

GoogleCodeExporter commented 9 years ago
Figured it out.  The new Browser seems to be picky about the Content Type and 
the
encoding declarations on the response header.

In EveMarketTool.PageServer, changed to add the content type and content length:

byte[] htmlData = System.Text.UTF8Encoding.UTF8.GetBytes(htmlOutput);
context.Response.ContentType = "text/html; charset=utf-8";
context.Response.ContentLength64 = htmlData.Length;
context.Response.OutputStream.Write(htmlData, 0, htmlData.Length);
context.Response.OutputStream.Close();

The welcome page now appears.  Seem to have a different problem, as I can't get 
past
the welcome page / trust page loop.

FWIW

Original comment by jlstro...@gmail.com on 13 Dec 2009 at 12:13

GoogleCodeExporter commented 9 years ago
Header keys have changed as well:

string systemName = context.Request.Headers["EVE_SOLARSYSTEMNAME"];
string charName = context.Request.Headers["EVE_CHARNAME"];
string charId = context.Request.Headers["EVE_CHARID"];

Original comment by jlstro...@gmail.com on 13 Dec 2009 at 7:58

GoogleCodeExporter commented 9 years ago
Nice work jlistrobel - if you send me a mail (my user name at gmail.com) I'll 
be happy to grant you svn commit 
access to the project, ao you can put these changes into the main branch for 
everyone to enjoy!

Original comment by mark.oco...@gmail.com on 13 Dec 2009 at 8:19

GoogleCodeExporter commented 9 years ago

Original comment by mark.oco...@gmail.com on 14 Dec 2009 at 7:10

GoogleCodeExporter commented 9 years ago
I've done a few modifications myself to make the information a little clearer 
in the
new browser, here's a screenshot:

http://img686.imageshack.us/img686/5826/navbotdominion.jpg

I've also changed the code to use the new CCPEVE.showInfo/setDestination so all 
that
works just fine and dandy now.  Hit me up if you'd like me to upload it 
somewhere or
whatever, as I'm not all that fond of subversion!

- Terry

Original comment by terry...@gmail.com on 16 Dec 2009 at 6:08

GoogleCodeExporter commented 9 years ago
I've also utilized the new IGB ability to request trust also:

http://img40.imageshack.us/img40/1683/navbottrust.jpg

Also added in the profit margin to the display, as well as a [Details] link 
that'll
open up the market details for that particular item, example below. :^)

http://img199.imageshack.us/img199/3521/navbotdominion2.jpg

Original comment by terry...@gmail.com on 16 Dec 2009 at 8:27

GoogleCodeExporter commented 9 years ago
That's some pretty amazing work! Very well done, indeed. A lot of people are 
waiting
for this update, and I am one of them (i've actually stopped trading and 
playing the
game for a while) so thank you. Please upload either the solution or the 
compiled
market tool dll (and maybe the exe) if you don't feel like submitting directly 
to the
svn, or you're just not used to working with one. I'm sure the other devs will 
gladly
update the svn themselves if you post the code. 

Once again, thanks!

Original comment by miske...@gmail.com on 16 Dec 2009 at 9:35

GoogleCodeExporter commented 9 years ago
Here you all go:

http://rapidshare.com/files/321814311/NavBot_J_Dominion_SrcAndBinary.rar.html
MD5: B3048A5912924C2160A72338EAB48A86 

Sorry about some of the HTML formatting and mess of comments everywhere in the 
code,
enjoy :^)

Original comment by terry...@gmail.com on 16 Dec 2009 at 9:53

GoogleCodeExporter commented 9 years ago
Another rapidshare link in case someone else still needs it
(only 10 downloads per file)

http://rapidshare.com/files/321844785/NavBot_J_Dominion_SrcAndBinary.rar.html
MD5: B3048A5912924C2160A72338EAB48A86 

Thanks for the great work guys

Original comment by Calvario...@gmail.com on 16 Dec 2009 at 11:27

GoogleCodeExporter commented 9 years ago
I just noticed the low security warning isn't exactly working as intended so 
use with
caution on that (the lowsec jump color is still shown) :^)

Original comment by terry...@gmail.com on 16 Dec 2009 at 11:46

GoogleCodeExporter commented 9 years ago
Very nice job works like a charm.  Now if only we can get it to scrub the cache 
like
that "other" program.  Nonetheless excellent job and I do appreciate the hard 
work
done by peeps much more intelligent than I.

Original comment by bjwim...@gmail.com on 16 Dec 2009 at 11:53

GoogleCodeExporter commented 9 years ago
To all those who made this solution possible, excellent work! Thank-you for 
your time
and effort, it is greatly appreciated!

Original comment by Archidd...@gmail.com on 17 Dec 2009 at 1:22

GoogleCodeExporter commented 9 years ago
Thanks for the excellent work on making it look better.  I have merged the 
changes
and checked them into SVN.  I also did a build and put it on the downloads 
list.  I
would be interested if anyone can test it and make sure it's working correctly. 
 I
tested on 64 bit Vista and 32 bit XP.  Thanks!

Original comment by jlstro...@gmail.com on 17 Dec 2009 at 5:22

GoogleCodeExporter commented 9 years ago

Original comment by jlstro...@gmail.com on 17 Dec 2009 at 5:22

GoogleCodeExporter commented 9 years ago
D'oh - jlstrobel, I didn't see your build up there. I've also updated the error 
messages and the QuickStartGuide to make the whole 'run as administrator' 
debacle a 
bit clearer. It should be easy enough to add permission elevation to NavBot in 
the 
future.

Thanks to everyone for the awesome contributions. If anyone wants SVN commit 
access, 
just email me at my id at gmail.com and I'll add you to the project.

jlstrobel: I replaced your download with a new one containing the updated error 
messages, rather than create yet another revision for them - sorry!

Original comment by mark.oco...@gmail.com on 17 Dec 2009 at 8:48