-
Notifications
You must be signed in to change notification settings - Fork 3
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
enable proxy for HttpClient #12
Conversation
Danke für den Fix - sieht erst einmal sinnvoll aus, auch wenn ich es nicht getestet habe. Den fehlerhaften Travis-Build schaue ich mir dann noch an, das hat zunächst nichts mit diesem PR zu tun. |
Hallo Friedrich,
freut mich, dass mein PR bei dir Anklang findet.
Ich nutze das Tool aus einem geschützten Netzwerk, da kam immer eine UnknownHostException. Mit der Änderung für den Proxy geht es. Allerdings muss man dafür das Java-Programm mit dem proxy-Parameter aufrufen. Das dürfte für den Standard-Nutzer bestimmt etwas kompliziert sein.
Ich bin begeisterter Geocacher auf GC und OC unter dem Namen salmiaker und ich kenne mich auch ein bisschen mit Java-Programmierung aus.
Ein paar Ideen zur Weiterentwicklung hätte ich noch:
- für Look&Feel das System-Look&Feel einstellen. Dann sieht das Programm ein bisschen angenehmer aus (wie ein normales Windows-Programm)
- den DuplicateDialog könnte man tabellarisch aufbauen (oder gibt es Fälle, wo mehrere Kandidaten für einen Cache existieren??) -> Tabellen haben den Vorteil der Sortierbarkeit
- für den DuplicateDialog Checkboxen hinzufügen, wo man mehrere/alle Logs auf einmal synchronisieren kann. Bisher muss man jede Zeile einzeln öffnen und das "Knöpfchen" drücken
- im DuplicateDialog auch den Status (aktiv/inaktiv/archiviert) der Caches anzeigen
- die Proxy-Einstellung im Einstellungsmenü anbieten
Grüße
Johannes
|
Zu den Ideen:
Für die Umsetzung gerne entsprechende Pull Requests erstellen. |
Hallo!
Ich habe mir grade mal die neuste Version vom cmanager gepullt.
Schön, dass es ein paar meiner Erweiterungen in den master geschafft haben. Dadurch kann ich die Anwendung nun auch hinter einem Proxy starten :-)
Mal noch eine Anregung:
Die "Spielerei" mit der Java Heap-Size würde ich entfernen ( ForkUtil.forkWithResizedHeapAndExit(arguments); )
Ich glaube das verursacht manchmal Probleme (Anwendung friert ein) und aus meiner Sicht gibt es auf modernen Rechnern keinen Grund für so eine Mini-Anwendung an der Java-Heapsize rum zu spielen.
Dann habe ich für meine aktuelle GC-Cache-Liste mal wieder eine Synchronisation durchgeführt.
Es gibt ein paar komische Effekte (s. Screenshot). Er findet unplausible Kandidaten und auf der anderen Seite findet er plausible Kandidaten NICHT :-/
Oder vielleicht ist einfach die Zuordnung durcheinander geraten, denn in Summe sind alle Kandiaten vorhanden...
Grüße
Johannes
Gesendet: Donnerstag, 28. Mai 2020 um 12:26 Uhr
Von: "FriedrichFroebel" <notifications@github.com>
An: "FriedrichFroebel/cmanager" <cmanager@noreply.github.com>
Cc: "jsteltze" <johannes.steltzer@gmx.net>, "Author" <author@noreply.github.com>
Betreff: Re: [FriedrichFroebel/cmanager] enable proxy for HttpClient (#12)
Zu den Ideen:
Für das Aussehen habe ich einen Issue erstellt. Ich selbst habe keine Probleme mit dem Standard-Java-Stil, da ich eh verschiedene Betriebssysteme verwende, aber es schadet auch nicht, das zu ändern.
Soweit ich das überblicke, sind mehrere Kandidaten möglich (wenn auch in den meisten Fällen eher unwahrscheinlich).
Die Massenübertragung wurde bereits im Opencaching-Forum diskutiert und wird es höchstwahrscheinlich nicht geben. Dem Benutzer soll bewusst sein, welche Caches er gerade loggt und dabei auch sicherstellen, dass es wirklich das richtige Doppellisting ist - eine automatisierte Übertragung kann dies nicht sicherstellen. Zudem sollte jenes Vorgehen die allgemeine Last auf die OKAPI besser verteilen.
Die beiden letzten Punkte habe ich als Issues angelegt.
Für die Umsetzung gerne entsprechende Pull Requests erstellen.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
Ich habe, wenn nicht unbedingt notwendig, keine größeren Umstellungen des Altcodes des Original-Upstreams von Samsung1 vorgenommen - abgesehen von Codestyle und Modulstruktur. Ob das wirklich Grund für die Freezes ist, kann ich nicht beurteilen - genauso wenig, inwieweit das in der Realität tatsächlich verwendet wird.
GitHub mag E-Mail-Bilder leider nicht, sodass das Bild fehlt. Es ist durchaus bekannt, dass plausible Kandidaten bei zu starker Abweichung der Daten weggefiltert werden. Ich habe schon ein paar kleinere Anpassungen vorgenommen, aber es wird immer Fehler geben - sowohl auf Benutzerseite als auch durch die verwendeten allgemeinen Heuristiken. Permutationen der Zuordnung sind mir nicht bekannt - und falls es an dem ist, dürfte das auf jeden Fall ein Bug sein. |
Den HttpClient ermöglichen einen Proxy zu nutzen, wenn das Programm in einem geschützten Netzwerk ausgeführt wird. Der Proxy wird aus den Java-Parametern ausgelesen (-DhttpsProxyHost=... -DhttpsProxyPort=...)