Modify

Opened 15 years ago

Closed 12 years ago

#4498 closed defect (fixed)

Can not upload GPS Track (OAuth)

Reported by: calibrator Owned by: team
Priority: normal Milestone:
Component: Plugin DirectUpload Version: latest
Keywords: Cc:

Description

LOSM 2959. Uploading GPS works OK with Normal_Authentication but impossible upload GPS track via plugin DirectUpload (v.19082) with OAuth (GPS Uploading granted in Oauth granted rights). See attache screens.

Attachments (3)

Settings_Oauth_Upload_GPS.PNG (23.2 KB ) - added by calibrator 15 years ago.
No passw_GPS_UPL_Oauth.PNG (9.9 KB ) - added by calibrator 15 years ago.
DirectUpload.patch (23.5 KB ) - added by ax 15 years ago.

Download all attachments as: .zip

Change History (19)

by calibrator, 15 years ago

by calibrator, 15 years ago

Attachment: No passw_GPS_UPL_Oauth.PNG added

comment:1 by stoecker, 15 years ago

Component: PluginPlugin directupload

comment:2 by stoecker, 15 years ago

Ticket #4933 has been marked as a duplicate of this ticket.

comment:3 by anonymous, 15 years ago

Summary: Can not upload GPS Track (OAuth)[PATCH] Can not upload GPS Track (OAuth)

find attached a patch that

  • adds oauth support,
  • adds tag, description, and privacy history and autocomplete to the upload dialog,
  • changes the way the gpx layer to be uploaded is selected among all open layers. you don't have to activate the gpx layer anymore, but just select it (as in "zoom to layer"). this saves some mouse clicks and doesn't disrupt the editing process (the current active editing layer can stay active)
  • (hopefully) simplifies some of the code

developed against r3627, updated to the latest r3648 without problems. i did some testing and found no problems - no guarantees though.

note that i had to use some not so pretty workarounds around josm architectural shortcomings. i hope i find the time to address those by providing patches to josm core which define better interfaces. i wouldn't mind if someone beats me to it, though :)

comment:4 by ax, 15 years ago

this last comment by "anonymous" is mine. is there an issue with the trac login? as soon as i press "preview" or "submit", i am logged out ...

comment:5 by stoecker, 15 years ago

Hmm, you're sure? I heard some of such issues, but could never reproduce them.

comment:6 by ax, 15 years ago

yep, definitely. i log in and get a "logged in as ax" on the top right. clicking any link or submitting any form or just reloading the page gives me the "Login" link on the top right again, and forms being submitted as anonymous. i have to manually fill the "Your email or username" box. both firefox and google chrome. are there different user classes?

comment:7 by stoecker, 15 years ago

You turned off cookies?

comment:8 by ax, 15 years ago

no. i have both a trac_session and a trac_form_token cookie. doesn't help.

comment:9 by stoecker, 15 years ago

Can you try another browser or disable Firefox plugins? Trac needs both the cookies as well as the form tokens in the HTML code to properly recognice a session. Please open a new report for this issue. If you switch to http or would also be nice when you could record data transfer using wireshark or similar tools and send it to me. You should have my mail from josm-dev.

comment:10 by ax, 15 years ago

the reason for the trac auth issue seems to have been determined.

regarding the patch for the actual gps track upload issue: can i do anything to get someone to look at (and commit) it ?

by ax, 15 years ago

Attachment: DirectUpload.patch added

comment:11 by ax, 15 years ago

removed an unnecessary comment and an unnecessary super. from the patch.

in reply to:  10 comment:12 by bastiK, 15 years ago

Replying to ax:

the reason for the trac auth issue seems to have been determined.

regarding the patch for the actual gps track upload issue: can i do anything to get someone to look at (and commit) it ?

I'll care for the patches tomorrow.

comment:13 by bastiK, 14 years ago

applied the patch as a temporary solution. (In [o24239])

comment:14 by bastiK, 14 years ago

Summary: [PATCH] Can not upload GPS Track (OAuth)Can not upload GPS Track (OAuth)

comment:15 by Don-vip, 12 years ago

OAuth upload works now.

comment:16 by Don-vip, 12 years ago

Resolution: fixed
Status: newclosed

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. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.