Modify

Opened 6 years ago

Closed 5 years ago

#8571 closed enhancement (fixed)

Enhance JOSM bug report system

Reported by: Don-vip Owned by: team
Priority: normal Milestone: 14.08
Component: Core bugreport Version:
Keywords: privacy Cc:

Description (last modified by Don-vip)

We have more and more bug reports without all the required information we need to reproduce it quickly or have a better understanding of it.

Users are often available to reply to our requests, but sometimes they are not. I'm sure there are some of "needinfo anonymous" tickets that could have been easily fixed if we had the required info.

Even when reporters willingly help us, it's often a loss of time, both for them and us.

I think of several enhancements that could help us to improve the situation when the user chooses to report a bug:

  • Increase the size limit of text that can be pasted to our Trac instance, in order to avoid incomplete stracktraces -> Done (r5827)
  • Add some wiki formatting to have nicer bug reports. -> Done (r7419)
  • Add how JOSM has been started (Webstart or not, command-line arguments) -> Done (r5831)
  • Automatically attach the commands stack that led to the JOSM bug
  • Automatically attach some of the JOSM preferences, to be discussed to avoid privacy concerns
  • Automatically export and attach active layers when the crash occurred (to be also discussed for privacy concerns). An external and configurable free file hosting service support may be implemented to upload large .osm files that could not be stored on this Trac instance. See #6804

What do you think ?

Attachments (1)

tinyurl.patch (2.5 KB) - added by Don-vip 6 years ago.
Fix for bug reports with incomplete stacktraces

Download all attachments as: .zip

Change History (27)

comment:1 Changed 6 years ago by skyper

+10

See also #2557, #6804 and #7708.

Please, no upload of privat data without extra confirmation.
I am not sure if we really need an external host. Usually, you only need a small file for demonstration and JOSM even supports gz and bz2.

Thought the limit was 4mb though.

comment:2 in reply to:  1 Changed 6 years ago by Don-vip

Replying to skyper:

Please, no upload of private data without extra confirmation.

Of course. We just need to define what is private data and what is public data.

I am not sure if we really need an external host. Usually, you only need a small file for demonstration and JOSM even supports gz and bz2.

I agree, but when a bug occurs on a large dataset, it would be very difficult (quite impossible ?) to automatically reduce it to the minimal dataset leading to the exception.

EDIT: thanks for the above tickets, I didn't think they already exist.

Last edited 6 years ago by Don-vip (previous) (diff)

comment:3 in reply to:  description ; Changed 6 years ago by Don-vip

Replying to Don-vip:

  • Increase the size limit of text that can be pasted to our Trac instance, in order to avoid incomplete stracktraces

Wow, I have found the reason of this. Nothing to do with Trac, neither Java or JOSM. There's a limitation in Windows that prevents us to open URLs longer than 2083 characters with the default browser (same behaviour observed with JOSM's PlatformHookWindows.openUrl() and Java's Desktop.getDesktop().browse(uri) ). It's even worse if the browser is Internet Explorer, the GET requests cannot exceed 2047 characters.

No idea how to fix this.

EDIT: the first suggested solution works if we use tinyurl to shorten URL (it's the only shortening service that works with this URL):

Original URL:

http://josm.openstreetmap.de/josmticket?tdata=QnVpbGQtRGF0ZTogMjAxMy0wNC0wMyAyMjozNzozNQpSZXZpc2lvbjogNTgyNApJcy1Mb2NhbC1CdWlsZDogdHJ1ZQoKSWRlbnRpZmljYXRpb246IEpPU00vMS41ICg1ODI0IFNWTiBlbl9HQikgV2luZG93cyA4Ck1lbW9yeSBVc2FnZTogMTI5IE1CIC8gMTgxMyBNQiAoNzQgTUIgYWxsb2NhdGVkLCBidXQgZnJlZSkKSmF2YSB2ZXJzaW9uOiAxLjYuMF80MywgU3VuIE1pY3Jvc3lzdGVtcyBJbmMuLCBKYXZhIEhvdFNwb3QoVE0pIDY0LUJpdCBTZXJ2ZXIgVk0KT3BlcmF0aW5nIHN5c3RlbTogV2luZG93cyA4CkRhdGFzZXQgY29uc2lzdGVuY3kgdGVzdDogTm8gcHJvYmxlbXMgZm91bmQKClBsdWdpbjogRGlyZWN0RG93bmxvYWQgKDI5MjIyKQpQbHVnaW46IERpcmVjdFVwbG9hZCAoMjkyMTApClBsdWdpbjogRml4QWRkcmVzc2VzICgyOTIxMCkKUGx1Z2luOiBIb3VzZU51bWJlclRhZ2dpbmdUb29sICgyOTIxMCkKUGx1Z2luOiBPcGVuaW5nSG91cnNFZGl0b3IgKDI5MjEwKQpQbHVnaW46IFJvYWRTaWducyAoMjk0MTcpClBsdWdpbjogVG9tYlBsdWdpbiAoNDEpClBsdWdpbjogYnVpbGRpbmdzX3Rvb2xzICgyOTIxMCkKUGx1Z2luOiBjYWRhc3RyZS1mciAoMjkyMTApClBsdWdpbjogY29udG91cm1lcmdlICgxMDAzKQpQbHVnaW46IGRvd25sb2FkX2Fsb25nICgyOTIxMCkKUGx1Z2luOiBleHRfdG9vbHMgKDI5MjEwKQpQbHVnaW46IGdlb3Rvb2xzICgyOTMxOSkKUGx1Z2luOiBncHhmaWx0ZXIgKDI5MjIyKQpQbHVnaW46IGltYWdlcnkteG1sLWJvdW5kcyAoMjk0NjEpClBsdWdpbjoganRzICgyOTMxOSkKUGx1Z2luOiBtZXJnZS1vdmVybGFwICgyOTIxMCkKUGx1Z2luOiBtaWNoaWdhbl9sZWZ0ICgyOTIxMCkKUGx1Z2luOiBtaXJyb3JlZF9kb3dubG9hZCAoMjkyMTApClBsdWdpbjogbXVsdGlwb2x5LWNvbnZlcnQgKDI5MjEwKQpQbHVnaW46IG9wZW5kYXRhICgyOTMwMSkKUGx1Z2luOiBvcGVudmlzaWJsZSAoMjkyMjIpClBsdWdpbjogcHJpbnQgKDI5MjEwKQpQbHVnaW46IHByb2o0aiAoMjkzMzIpClBsdWdpbjogcHVibGljX3RyYW5zcG9ydCAoMjkyMjIpClBsdWdpbjogcmVsdG9vbGJveCAoMjk0NTkpClBsdWdpbjogcmV2ZXJ0ZXIgKDI5MzU5KQpQbHVnaW46IHRhZzJsaW5rICgyOTQ2MSkKUGx1Z2luOiB0dXJubGFuZXMgKDI5MjEwKQpQbHVnaW46IHR1cm5yZXN0cmljdGlvbnMgKDI5MjEwKQpQbHVnaW46IHV0aWxzcGx1Z2luMiAoMjkzOTIpClBsdWdpbjogd2lraXBlZGlhICgyOTMzMCkKCmphdmEubGFuZy5OdWxsUG9pbnRlckV4Y2VwdGlvbgoJYXQgb3JnLm9wZW5zdHJlZXRtYXAuam9zbS5ndWkuZGlhbG9ncy5WYWxpZGF0b3JEaWFsb2ckQ2xpY2tXYXRjaC5tb3VzZVByZXNzZWQoVmFsaWRhdG9yRGlhbG9nLmphdmE6NDc4KQoJYXQgamF2YS5hd3QuQVdURXZlbnRNdWx0aWNhc3Rlci5tb3VzZVByZXNzZWQoQVdURXZlbnRNdWx0aWNhc3Rlci5qYXZhOjI2MykKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5wcm9jZXNzTW91c2VFdmVudChDb21wb25lbnQuamF2YTo2Mjk0KQoJYXQgamF2YXguc3dpbmcuSkNvbXBvbmVudC5wcm9jZXNzTW91c2VFdmVudChKQ29tcG9uZW50LmphdmE6MzI3NSkKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5wcm9jZXNzRXZlbnQoQ29tcG9uZW50LmphdmE6NjA2MikKCWF0IGphdmEuYXd0LkNvbnRhaW5lci5wcm9jZXNzRXZlbnQoQ29udGFpbmVyLmphdmE6MjAzOSkKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5kaXNwYXRjaEV2ZW50SW1wbChDb21wb25lbnQuamF2YTo0NjYwKQoJYXQgamF2YS5hd3QuQ29udGFpbmVyLmRpc3BhdGNoRXZlbnRJbXBsKENvbnRhaW5lci5qYXZhOjIwOTcpCglhdCBqYXZhLmF3dC5Db21wb25lbnQuZGlzcGF0Y2hFdmVudChDb21wb25lbnQuamF2YTo0NDg4KQoJYXQgamF2YS5hd3QuTGlnaHR3ZWlnaHREaXNwYXRjaGVyLnJldGFyZ2V0TW91c2VFdmVudChDb250YWluZXIuamF2YTo0NTc1KQoJYXQgamF2YS5hd3QuTGlnaHR3ZWlnaHREaXNwYXRjaGVyLnByb2Nlc3NNb3VzZUV2ZW50KENvbnRhaW5lci5qYXZhOjQyMzMpCglhdCBqYXZhLmF3dC5MaWdodHdlaWdodERpc3BhdGNoZXIuZGlzcGF0Y2hFdmVudChDb250YWluZXIuamF2YTo0MTY2KQoJYXQgamF2YS5hd3QuQ29udGFpbmVyLmRpc3BhdGNoRXZlbnRJbXBsKENvbnRhaW5lci5qYXZhOjIwODMpCglhdCBqYXZhLmF3dC5XaW5kb3cuZGlzcGF0Y2hFdmVudEltcGwoV2luZG93LmphdmE6MjQ4OSkKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5kaXNwYXRjaEV2ZW50KENvbXBvbmVudC5qYXZhOjQ0ODgpCglhdCBqYXZhLmF3dC5FdmVudFF1ZXVlLmRpc3BhdGNoRXZlbnRJbXBsKEV2ZW50UXVldWUuamF2YTo2NzQpCglhdCBqYXZhLmF3dC5FdmVudFF1ZXVlLmFjY2VzcyQ0MDAoRXZlbnRRdWV1ZS5qYXZhOjgxKQoJYXQgamF2YS5hd3QuRXZlbnRRdWV1ZSQyLnJ1bihFdmVudFF1ZXVlLmphdmE6NjMzKQoJYXQgamF2YS5hd3QuRXZlbnRRdWV1ZSQyLnJ1bihFdmVudFF1ZXVlLmphdmE6NjMxKQoJYXQgamF2YS5zZWN1cml0eS5BY2Nlc3NDb250cm9sbGVyLmRvUHJpdmlsZWdlZChOYXRpdmUgTWV0aG9kKQoJYXQgamF2YS5zZWN1cml0eS5BY2Nlc3NDb250cm9sQ29udGV4dCQxLmRvSW50ZXJzZWN0aW9uUHJpdmlsZWdlKEFjY2Vzc0NvbnRyb2xDb250ZXh0LmphdmE6ODcpCglhdCBqYXZhLnNlY3VyaXR5LkFjY2Vzc0NvbnRyb2xDb250ZXh0JDEuZG9JbnRlcnNlY3Rpb25Qcml2aWxlZ2UoQWNjZXNzQ29udHJvbENvbnRleHQuamF2YTo5OCkKCWF0IGphdmEuYXd0LkV2ZW50UXVldWUkMy5ydW4oRXZlbnRRdWV1ZS5qYXZhOjY0NykKCWF0IGphdmEuYXd0LkV2ZW50UXVldWUkMy5ydW4oRXZlbnRRdWV1ZS5qYXZhOjY0NSkKCWF0IGphdmEuc2VjdXJpdHkuQWNjZXNzQ29udHJvbGxlci5kb1ByaXZpbGVnZWQoTmF0aXZlIE1ldGhvZCkKCWF0IGphdmEuc2VjdXJpdHkuQWNjZXNzQ29udHJvbENvbnRleHQkMS5kb0ludGVyc2VjdGlvblByaXZpbGVnZShBY2Nlc3NDb250cm9sQ29udGV4dC5qYXZhOjg3KQoJYXQgamF2YS5hd3QuRXZlbnRRdWV1ZS5kaXNwYXRjaEV2ZW50KEV2ZW50UXVldWUuamF2YTo2NDQpCglhdCBqYXZhLmF3dC5FdmVudERpc3BhdGNoVGhyZWFkLnB1bXBPbmVFdmVudEZvckZpbHRlcnMoRXZlbnREaXNwYXRjaFRocmVhZC5qYXZhOjI2OSkKCWF0IGphdmEuYXd0LkV2ZW50RGlzcGF0Y2hUaHJlYWQucHVtcEV2ZW50c0ZvckZpbHRlcihFdmVudERpc3BhdGNoVGhyZWFkLmphdmE6MTg0KQoJYXQgamF2YS5hd3QuRXZlbnREaXNwYXRjaFRocmVhZC5wdW1wRXZlbnRzRm9ySGllcmFyY2h5KEV2ZW50RGlzcGF0Y2hUaHJlYWQuamF2YToxNzQpCglhdCBqYXZhLmF3dC5FdmVudERpc3BhdGNoVGhyZWFkLnB1bXBFdmVudHMoRXZlbnREaXNwYXRjaFRocmVhZC5qYXZhOjE2OSkKCWF0IGphdmEuYXd0LkV2ZW50RGlzcGF0Y2hUaHJlYWQucHVtcEV2ZW50cyhFdmVudERpc3BhdGNoVGhyZWFkLmphdmE6MTYxKQoJYXQgamF2YS5hd3QuRXZlbnREaXNwYXRjaFRocmVhZC5ydW4oRXZlbnREaXNwYXRjaFRocmVhZC5qYXZhOjEyMikK

Tiny URL:

http://tinyurl.com/caw7ubh

It works with Chrome 26, Firefox 20, and strangely, even with IE10.

Last edited 6 years ago by Don-vip (previous) (diff)

Changed 6 years ago by Don-vip

Attachment: tinyurl.patch added

Fix for bug reports with incomplete stacktraces

comment:4 Changed 6 years ago by Don-vip

Summary: Enhance JOSM bug report system[Patch] Enhance JOSM bug report system

comment:5 in reply to:  3 ; Changed 6 years ago by bastiK

Replying to Don-vip:

Replying to Don-vip:

  • Increase the size limit of text that can be pasted to our Trac instance, in order to avoid incomplete stracktraces

Wow, I have found the reason of this. Nothing to do with Trac, neither Java or JOSM. There's a limitation in Windows that prevents us to open URLs longer than 2083 characters with the default browser (same behaviour observed with JOSM's PlatformHookWindows.openUrl() and Java's Desktop.getDesktop().browse(uri) ). It's even worse if the browser is Internet Explorer, the GET requests cannot exceed 2047 characters.

No idea how to fix this.

EDIT: the first suggested solution works if we use tinyurl to shorten URL (it's the only shortening service that works with this URL):

Original URL:

http://josm.openstreetmap.de/josmticket?tdata=QnVpbGQtRGF0ZTogMjAxMy0wNC0wMyAyMjozNzozNQpSZXZpc2lvbjogNTgyNApJcy1Mb2NhbC1CdWlsZDogdHJ1ZQoKSWRlbnRpZmljYXRpb246IEpPU00vMS41ICg1ODI0IFNWTiBlbl9HQikgV2luZG93cyA4Ck1lbW9yeSBVc2FnZTogMTI5IE1CIC8gMTgxMyBNQiAoNzQgTUIgYWxsb2NhdGVkLCBidXQgZnJlZSkKSmF2YSB2ZXJzaW9uOiAxLjYuMF80MywgU3VuIE1pY3Jvc3lzdGVtcyBJbmMuLCBKYXZhIEhvdFNwb3QoVE0pIDY0LUJpdCBTZXJ2ZXIgVk0KT3BlcmF0aW5nIHN5c3RlbTogV2luZG93cyA4CkRhdGFzZXQgY29uc2lzdGVuY3kgdGVzdDogTm8gcHJvYmxlbXMgZm91bmQKClBsdWdpbjogRGlyZWN0RG93bmxvYWQgKDI5MjIyKQpQbHVnaW46IERpcmVjdFVwbG9hZCAoMjkyMTApClBsdWdpbjogRml4QWRkcmVzc2VzICgyOTIxMCkKUGx1Z2luOiBIb3VzZU51bWJlclRhZ2dpbmdUb29sICgyOTIxMCkKUGx1Z2luOiBPcGVuaW5nSG91cnNFZGl0b3IgKDI5MjEwKQpQbHVnaW46IFJvYWRTaWducyAoMjk0MTcpClBsdWdpbjogVG9tYlBsdWdpbiAoNDEpClBsdWdpbjogYnVpbGRpbmdzX3Rvb2xzICgyOTIxMCkKUGx1Z2luOiBjYWRhc3RyZS1mciAoMjkyMTApClBsdWdpbjogY29udG91cm1lcmdlICgxMDAzKQpQbHVnaW46IGRvd25sb2FkX2Fsb25nICgyOTIxMCkKUGx1Z2luOiBleHRfdG9vbHMgKDI5MjEwKQpQbHVnaW46IGdlb3Rvb2xzICgyOTMxOSkKUGx1Z2luOiBncHhmaWx0ZXIgKDI5MjIyKQpQbHVnaW46IGltYWdlcnkteG1sLWJvdW5kcyAoMjk0NjEpClBsdWdpbjoganRzICgyOTMxOSkKUGx1Z2luOiBtZXJnZS1vdmVybGFwICgyOTIxMCkKUGx1Z2luOiBtaWNoaWdhbl9sZWZ0ICgyOTIxMCkKUGx1Z2luOiBtaXJyb3JlZF9kb3dubG9hZCAoMjkyMTApClBsdWdpbjogbXVsdGlwb2x5LWNvbnZlcnQgKDI5MjEwKQpQbHVnaW46IG9wZW5kYXRhICgyOTMwMSkKUGx1Z2luOiBvcGVudmlzaWJsZSAoMjkyMjIpClBsdWdpbjogcHJpbnQgKDI5MjEwKQpQbHVnaW46IHByb2o0aiAoMjkzMzIpClBsdWdpbjogcHVibGljX3RyYW5zcG9ydCAoMjkyMjIpClBsdWdpbjogcmVsdG9vbGJveCAoMjk0NTkpClBsdWdpbjogcmV2ZXJ0ZXIgKDI5MzU5KQpQbHVnaW46IHRhZzJsaW5rICgyOTQ2MSkKUGx1Z2luOiB0dXJubGFuZXMgKDI5MjEwKQpQbHVnaW46IHR1cm5yZXN0cmljdGlvbnMgKDI5MjEwKQpQbHVnaW46IHV0aWxzcGx1Z2luMiAoMjkzOTIpClBsdWdpbjogd2lraXBlZGlhICgyOTMzMCkKCmphdmEubGFuZy5OdWxsUG9pbnRlckV4Y2VwdGlvbgoJYXQgb3JnLm9wZW5zdHJlZXRtYXAuam9zbS5ndWkuZGlhbG9ncy5WYWxpZGF0b3JEaWFsb2ckQ2xpY2tXYXRjaC5tb3VzZVByZXNzZWQoVmFsaWRhdG9yRGlhbG9nLmphdmE6NDc4KQoJYXQgamF2YS5hd3QuQVdURXZlbnRNdWx0aWNhc3Rlci5tb3VzZVByZXNzZWQoQVdURXZlbnRNdWx0aWNhc3Rlci5qYXZhOjI2MykKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5wcm9jZXNzTW91c2VFdmVudChDb21wb25lbnQuamF2YTo2Mjk0KQoJYXQgamF2YXguc3dpbmcuSkNvbXBvbmVudC5wcm9jZXNzTW91c2VFdmVudChKQ29tcG9uZW50LmphdmE6MzI3NSkKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5wcm9jZXNzRXZlbnQoQ29tcG9uZW50LmphdmE6NjA2MikKCWF0IGphdmEuYXd0LkNvbnRhaW5lci5wcm9jZXNzRXZlbnQoQ29udGFpbmVyLmphdmE6MjAzOSkKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5kaXNwYXRjaEV2ZW50SW1wbChDb21wb25lbnQuamF2YTo0NjYwKQoJYXQgamF2YS5hd3QuQ29udGFpbmVyLmRpc3BhdGNoRXZlbnRJbXBsKENvbnRhaW5lci5qYXZhOjIwOTcpCglhdCBqYXZhLmF3dC5Db21wb25lbnQuZGlzcGF0Y2hFdmVudChDb21wb25lbnQuamF2YTo0NDg4KQoJYXQgamF2YS5hd3QuTGlnaHR3ZWlnaHREaXNwYXRjaGVyLnJldGFyZ2V0TW91c2VFdmVudChDb250YWluZXIuamF2YTo0NTc1KQoJYXQgamF2YS5hd3QuTGlnaHR3ZWlnaHREaXNwYXRjaGVyLnByb2Nlc3NNb3VzZUV2ZW50KENvbnRhaW5lci5qYXZhOjQyMzMpCglhdCBqYXZhLmF3dC5MaWdodHdlaWdodERpc3BhdGNoZXIuZGlzcGF0Y2hFdmVudChDb250YWluZXIuamF2YTo0MTY2KQoJYXQgamF2YS5hd3QuQ29udGFpbmVyLmRpc3BhdGNoRXZlbnRJbXBsKENvbnRhaW5lci5qYXZhOjIwODMpCglhdCBqYXZhLmF3dC5XaW5kb3cuZGlzcGF0Y2hFdmVudEltcGwoV2luZG93LmphdmE6MjQ4OSkKCWF0IGphdmEuYXd0LkNvbXBvbmVudC5kaXNwYXRjaEV2ZW50KENvbXBvbmVudC5qYXZhOjQ0ODgpCglhdCBqYXZhLmF3dC5FdmVudFF1ZXVlLmRpc3BhdGNoRXZlbnRJbXBsKEV2ZW50UXVldWUuamF2YTo2NzQpCglhdCBqYXZhLmF3dC5FdmVudFF1ZXVlLmFjY2VzcyQ0MDAoRXZlbnRRdWV1ZS5qYXZhOjgxKQoJYXQgamF2YS5hd3QuRXZlbnRRdWV1ZSQyLnJ1bihFdmVudFF1ZXVlLmphdmE6NjMzKQoJYXQgamF2YS5hd3QuRXZlbnRRdWV1ZSQyLnJ1bihFdmVudFF1ZXVlLmphdmE6NjMxKQoJYXQgamF2YS5zZWN1cml0eS5BY2Nlc3NDb250cm9sbGVyLmRvUHJpdmlsZWdlZChOYXRpdmUgTWV0aG9kKQoJYXQgamF2YS5zZWN1cml0eS5BY2Nlc3NDb250cm9sQ29udGV4dCQxLmRvSW50ZXJzZWN0aW9uUHJpdmlsZWdlKEFjY2Vzc0NvbnRyb2xDb250ZXh0LmphdmE6ODcpCglhdCBqYXZhLnNlY3VyaXR5LkFjY2Vzc0NvbnRyb2xDb250ZXh0JDEuZG9JbnRlcnNlY3Rpb25Qcml2aWxlZ2UoQWNjZXNzQ29udHJvbENvbnRleHQuamF2YTo5OCkKCWF0IGphdmEuYXd0LkV2ZW50UXVldWUkMy5ydW4oRXZlbnRRdWV1ZS5qYXZhOjY0NykKCWF0IGphdmEuYXd0LkV2ZW50UXVldWUkMy5ydW4oRXZlbnRRdWV1ZS5qYXZhOjY0NSkKCWF0IGphdmEuc2VjdXJpdHkuQWNjZXNzQ29udHJvbGxlci5kb1ByaXZpbGVnZWQoTmF0aXZlIE1ldGhvZCkKCWF0IGphdmEuc2VjdXJpdHkuQWNjZXNzQ29udHJvbENvbnRleHQkMS5kb0ludGVyc2VjdGlvblByaXZpbGVnZShBY2Nlc3NDb250cm9sQ29udGV4dC5qYXZhOjg3KQoJYXQgamF2YS5hd3QuRXZlbnRRdWV1ZS5kaXNwYXRjaEV2ZW50KEV2ZW50UXVldWUuamF2YTo2NDQpCglhdCBqYXZhLmF3dC5FdmVudERpc3BhdGNoVGhyZWFkLnB1bXBPbmVFdmVudEZvckZpbHRlcnMoRXZlbnREaXNwYXRjaFRocmVhZC5qYXZhOjI2OSkKCWF0IGphdmEuYXd0LkV2ZW50RGlzcGF0Y2hUaHJlYWQucHVtcEV2ZW50c0ZvckZpbHRlcihFdmVudERpc3BhdGNoVGhyZWFkLmphdmE6MTg0KQoJYXQgamF2YS5hd3QuRXZlbnREaXNwYXRjaFRocmVhZC5wdW1wRXZlbnRzRm9ySGllcmFyY2h5KEV2ZW50RGlzcGF0Y2hUaHJlYWQuamF2YToxNzQpCglhdCBqYXZhLmF3dC5FdmVudERpc3BhdGNoVGhyZWFkLnB1bXBFdmVudHMoRXZlbnREaXNwYXRjaFRocmVhZC5qYXZhOjE2OSkKCWF0IGphdmEuYXd0LkV2ZW50RGlzcGF0Y2hUaHJlYWQucHVtcEV2ZW50cyhFdmVudERpc3BhdGNoVGhyZWFkLmphdmE6MTYxKQoJYXQgamF2YS5hd3QuRXZlbnREaXNwYXRjaFRocmVhZC5ydW4oRXZlbnREaXNwYXRjaFRocmVhZC5qYXZhOjEyMikK

Tiny URL:

http://tinyurl.com/caw7ubh

It works with Chrome 26, Firefox 20, and strangely, even with IE10.

o_O There surely must be another way. :)

comment:6 in reply to:  5 Changed 6 years ago by Don-vip

Replying to bastiK:

o_O There surely must be another way. :)

Yes :) Another method, without any external service, is to compress the stacktrace before encoding it in Base64. With the previously given example, I have the following results with GZIPOutputStream and ZipOutputStream:

Data: 3543 characters
Base64: 4724 characters -> KO
GZIP: 1104 bytes
GZIP + Base64: 1512 characters -> OK
ZIP: 1210 bytes
ZIP + Base64: 1660 characters -> OK

So GZIP is the best choice:

                        ByteArrayOutputStream out = new ByteArrayOutputStream();
                        GZIPOutputStream gzip = new GZIPOutputStream(out);
                        gzip.write(data.getBytes("UTF-8"));
                        gzip.close();
                        String gzdata = Base64.encode(out.toString(), true);
Last edited 6 years ago by Don-vip (previous) (diff)

comment:7 Changed 6 years ago by bastiK

Maybe a small cgi-script on the server, that reads POST data and redirects the user to the trac URL with GET parameter?

comment:8 in reply to:  7 ; Changed 6 years ago by stoecker

Replying to bastiK:

Maybe a small cgi-script on the server, that reads POST data and redirects the user to the trac URL with GET parameter?

What do you think does /josmticket do? We need to add decompression only :-)

Making it gdata as argument for compressed data.

comment:9 Changed 6 years ago by Don-vip

In 5827/josm:

see #8571 - Compress bug reports before encoding them to have shorter URLs (to avoid incomplete stacktraces on Windows systems)

comment:10 in reply to:  8 Changed 6 years ago by Don-vip

Description: modified (diff)
Summary: [Patch] Enhance JOSM bug report systemEnhance JOSM bug report system

Replying to stoecker:

We need to add decompression only :-)

Not as easy as I thought, but done :)

comment:11 Changed 6 years ago by Don-vip

Description: modified (diff)

comment:12 Changed 6 years ago by Don-vip

Description: modified (diff)

comment:13 Changed 6 years ago by Don-vip

In 5831/josm:

see #8571 - Include command-line arguments in status report

comment:14 Changed 6 years ago by Don-vip

Description: modified (diff)

comment:15 in reply to:  8 Changed 6 years ago by bastiK

Replying to stoecker:

Replying to bastiK:

Maybe a small cgi-script on the server, that reads POST data and redirects the user to the trac URL with GET parameter?

What do you think does /josmticket do? We need to add decompression only :-)

Making it gdata as argument for compressed data.

Ok, I get it, you cannot open POST request in a browser window. Providing a service like tinyurl on our server for this sole purpose would be quite bothersome, so let's hope the compression is enough for now!

comment:16 Changed 6 years ago by Don-vip

Description: modified (diff)

comment:17 Changed 6 years ago by Don-vip

In 5833/josm:

see #8571 - Small improvements to status report

comment:18 Changed 6 years ago by Don-vip

Description: modified (diff)

comment:19 Changed 6 years ago by Don-vip

In 5840/josm:

see #8571, #8593 - Hide values of proxy VM arguments for privacy concerns

comment:20 Changed 6 years ago by Don-vip

In 5849/josm:

fix #8584 - see #8571 : Add "Report bug" button in Status report dialog, change bug report link in About dialog

comment:21 Changed 6 years ago by Don-vip

In 5851/josm:

see #8602, #8571 : use detailed OS info in status report

comment:22 Changed 6 years ago by Don-vip

In 5873/josm:

see #8571, #8602, #8606: Remove "Operating system" line from status report header as it is now fully included in Agent string

comment:23 Changed 6 years ago by Manu1400

Keywords: privacy added

comment:24 Changed 5 years ago by Don-vip

Component: CoreCore bugreport
Keywords: bug report removed

comment:25 Changed 5 years ago by Don-vip

Description: modified (diff)

comment:26 Changed 5 years ago by Don-vip

Milestone: 14.08
Resolution: fixed
Status: newclosed

This ticket is now fixed with r7419. I won't implement other points, or with another ticket (#6804).

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment


E-mail address and name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.