Requests.get failing when using inside function

Question

I am using python requests library to get the data from adzuna api .

if i try

r = requests.get("http://api.adzuna.com/v1/api/jobs/gb/search/1?app_id=appID\
    &app_key=appKEY&results_per_page=50&\
    what=entry%20level&content-type=application/json")
print r.text

it is fetching me data But if i wrap this inside a fuction

def getData ():
    r = requests.get("http://api.adzuna.com/v1/api/jobs/gb/search/1?app_id=appID\
    &app_key=appKey&results_per_page=50&\
    what=entry%20level&content-type=application/json")
    print r.text

getData()

it is giving me following exception

{"display":"Authorisation failed","__CLASS__":"Adzuna::API::Response::Exception","doc":"http://api.adzuna.com/v1/doc","exception":"AUTH_FAIL"}

what is the wrong with this function ?

See the Image for more info

Hi, thanks for pointing it out, it is copy paste mistake, i edited queston — anilkunchalaece, Jan 11 '18 at 08:15
@cᴏʟᴅsᴘᴇᴇᴅ not a duplicate, at least not a duplicate of this question. — t.m.adam, Jan 11 '18 at 08:17
@t.m.adam Really? Won't converting `print(r.text)` to `return r.text` solve OP's problem? — cs95, Jan 11 '18 at 08:18
@COLDSPEED why it is duplicate ? It is entirely different question - if not please explain — anilkunchalaece, Jan 11 '18 at 08:19
@cᴏʟᴅsᴘᴇᴇᴅ OP gets a response anyway, but it's en error response. — t.m.adam, Jan 11 '18 at 08:20
@anilkunchala i don't see why the 2nd case returns a different response. Please make sure that you have the same url or any other parameters. — t.m.adam, Jan 11 '18 at 08:29
seems like you need an app_key and app_id in order to make api request — ddor254, Jan 11 '18 at 09:06
@t.m.adam both url's are same , Do you think the problem is with AUTH ? i tried to pass them as headers - but it's not successful. — anilkunchalaece, Jan 11 '18 at 11:59
Both urls look identical, so the response should be the same. The only think that think of is the line breaks. Can you try again with the url in a single line? — t.m.adam, Jan 11 '18 at 13:24
thanks, can you make that as answer ? so i can mark it as answered ? — anilkunchalaece, Jan 12 '18 at 05:50
You could accept @NathanVērzemnieks answer bellow, it's the proper way to request GET parameters. — t.m.adam, Jan 12 '18 at 10:54

score 0 · Answer 1 · answered Jan 11 '18 at 08:37

0

In the second url you have a sapce between gb/search/ and the 1 after him.

answered Jan 11 '18 at 08:37

ddor254

1,570
1
12
28

thanks for noticing (edited question)- that's a another copy paste error. the url is working but authorization is failing i.e getting an AUTH_FAIL Exception – anilkunchalaece Jan 11 '18 at 08:51

Nathan Vērzemnieks · Accepted Answer · 2018-01-11T17:08:38.133

0

(edited) Since using a single-line query fixed your problem, I'll recommend that you don't try to build the query yourself but instead let requests do it for you, like so:

query = {'app_id': 'appID',
         'app_key': 'appKey',
         'content-type': 'application/json',
         'results_per_page': '50',
         'what': 'entry level'}
r = requests.get('http://api.adzuna.com/v1/api/jobs/gb/search/1', params=query)

This should be less error-prone and is certainly more convenient.

edited Jan 11 '18 at 17:08

answered Jan 11 '18 at 08:38

Nathan Vērzemnieks

5,495
1
11
23

Thanks for noticing, that's just a copy paste mistake - i edited question – anilkunchalaece Jan 11 '18 at 08:52

score 0 · Answer 3 · edited Sep 24 '21 at 17:53

I faced the same problem with a POST call and learned that problem was with formatting the text I was passing, some APIs can't handle any change on the format. inside function you have extra indentation and that messes up what I was posting. Instead of passing simple string:

Pass parameters in a json format if possible
Have all in one line
Reformat and print the string and make sure they look exactly the same in and out of function.

Requests.get failing when using inside function

3 Answers3