3meters / proxibase

Patchr web service
1 stars 0 forks source link

404 Error trying to delete a user #412

Closed Jaymassena closed 8 years ago

Jaymassena commented 8 years ago

This is from the Android client. Everything looks right to me for the call.

==== Req: 1462977, from: ::ffff:97.126.94.153, worker: 2, on: Sat, 09 Apr 2016 15:36:55 GMT post /v1/user/us.160409.55327.661.743423?erase=true { user: 'us.160409.55327.661.743423', session: 'ca84cfe72d96bb9d7e73062b6fb5e25ded82178f' } ==== Res: 1462977, time: 6, statusCode: 404

Jaymassena commented 8 years ago

I was able to delete the user from the iOS client. Does the http verb matter here? DELETE vs POST.

==== Req: 15832941, from: ::ffff:97.126.94.153, worker: 1, on: Sat, 09 Apr 2016 16:00:12 GMT delete /v1/user/us.160409.55327.661.743423?erase=true&session=24232e656cc95b31a8ac8a17fd8a8153f6bfb052&user=us.160409.55327.661.743423 ==== Res: 15832941, time: 98, statusCode: 200

georgesnelling commented 8 years ago

yes, the http verb must be delete.

On Sat, Apr 9, 2016 at 9:02 AM, Jay Massena notifications@github.com wrote:

I was able to delete the user from the iOS client. Does the http verb matter here? DELETE vs POST.

==== Req: 15832941, from: ::ffff:97.126.94.153, worker: 1, on: Sat, 09 Apr 2016 16:00:12 GMT delete /v1/user/us.160409.55327.661.743423?erase=true&session=24232e656cc95b31a8ac8a17fd8a8153f6bfb052&user=us.160409.55327.661.743423 ==== Res: 15832941, time: 98, statusCode: 200

— You are receiving this because you were assigned. Reply to this email directly or view it on GitHub https://github.com/3meters/proxibase/issues/412#issuecomment-207812657

Jaymassena commented 8 years ago

Can a delete still include a post body? Or do I have to put the session/user in the query string.

-J-

From: George Snelling Reply-To: 3meters/proxibase Date: Saturday, April 9, 2016 at 9:39 AM To: 3meters/proxibase Cc: "jay@3meters.commailto:jay@3meters.com" Subject: Re: [3meters/proxibase] 404 Error trying to delete a user (#412)

yes, the http verb must be delete.

On Sat, Apr 9, 2016 at 9:02 AM, Jay Massena notifications@github.com<mailto:notifications@github.com> wrote:

I was able to delete the user from the iOS client. Does the http verb matter here? DELETE vs POST.

==== Req: 15832941, from: ::ffff:97.126.94.153, worker: 1, on: Sat, 09 Apr 2016 16:00:12 GMT delete /v1/user/us.160409.55327.661.743423?erase=true&session=24232e656cc95b31a8ac8a17fd8a8153f6bfb052&user=us.160409.55327.661.743423 ==== Res: 15832941, time: 98, statusCode: 200

— You are receiving this because you were assigned. Reply to this email directly or view it on GitHub https://github.com/3meters/proxibase/issues/412#issuecomment-207812657

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHubhttps://github.com/3meters/proxibase/issues/412#issuecomment-207816667

georgesnelling commented 8 years ago

query string is HTTP standard. (deletes and gets should ignore the body altogether), but let me check to see if I did something different. hold on a sec..

On Sat, Apr 9, 2016 at 9:45 AM, Jay Massena notifications@github.com wrote:

Can a delete still include a post body? Or do I have to put the session/user in the query string.

-J-

From: George Snelling Reply-To: 3meters/proxibase Date: Saturday, April 9, 2016 at 9:39 AM To: 3meters/proxibase Cc: "jay@3meters.commailto:jay@3meters.com" Subject: Re: [3meters/proxibase] 404 Error trying to delete a user (#412)

yes, the http verb must be delete.

On Sat, Apr 9, 2016 at 9:02 AM, Jay Massena <notifications@github.com mailto:notifications@github.com> wrote:

I was able to delete the user from the iOS client. Does the http verb matter here? DELETE vs POST.

==== Req: 15832941, from: ::ffff:97.126.94.153, worker: 1, on: Sat, 09 Apr 2016 16:00:12 GMT delete

/v1/user/us.160409.55327.661.743423?erase=true&session=24232e656cc95b31a8ac8a17fd8a8153f6bfb052&user=us.160409.55327.661.743423

==== Res: 15832941, time: 98, statusCode: 200

— You are receiving this because you were assigned. Reply to this email directly or view it on GitHub https://github.com/3meters/proxibase/issues/412#issuecomment-207812657

— You are receiving this because you authored the thread. Reply to this email directly or view it on GitHub< https://github.com/3meters/proxibase/issues/412#issuecomment-207816667>

— You are receiving this because you were assigned. Reply to this email directly or view it on GitHub https://github.com/3meters/proxibase/issues/412#issuecomment-207817106

Jaymassena commented 8 years ago

Looks like I'm using the query string for the iOS client so we should just make it the same.

georgesnelling commented 8 years ago

yeah, i only parse the body for posts and puts, which sticks to the http spec.

On Sat, Apr 9, 2016 at 9:54 AM, Jay Massena notifications@github.com wrote:

Looks like I'm using the query string for the iOS client so we should just make it the same.

— You are receiving this because you were assigned. Reply to this email directly or view it on GitHub https://github.com/3meters/proxibase/issues/412#issuecomment-207817563

georgesnelling commented 8 years ago

fwiw, most people now agree it was a mistake in the spec, because big complicated query strings are totally unreadable.

On Sat, Apr 9, 2016 at 9:55 AM, George Snelling georgesn@gmail.com wrote:

yeah, i only parse the body for posts and puts, which sticks to the http spec.

On Sat, Apr 9, 2016 at 9:54 AM, Jay Massena notifications@github.com wrote:

Looks like I'm using the query string for the iOS client so we should just make it the same.

— You are receiving this because you were assigned. Reply to this email directly or view it on GitHub https://github.com/3meters/proxibase/issues/412#issuecomment-207817563

Jaymassena commented 8 years ago

OK, I got this working correctly now using the DELETE verb and credentials in the query string.

Jaymassena commented 8 years ago

I like short clean urls :-)