C#•3y ago

❔ HttpClient exiting program during GetStringAsync() and GetAsync()

I'm sending a GET to an API, but when debugging, as soon as you step off the last line, it cuts out the program without returning any value. Does anyone have any ideas?

Angius•2/3/23, 12:52 AM

Try wrapping it in a and see if there's some exception happening?

AAngius Try wrapping it in a `try/catch` and see if there's some exception happening?

Japemaster BradOP•2/3/23, 12:57 AM

No exception thrown, just a clean exit. I've come across this before, I think it was something about using .GetAwaiter().GetResult(), but that doesn't seem to give me much

AAngius Try wrapping it in a `try/catch` and see if there's some exception happening?

Japemaster BradOP•2/3/23, 1:00 AM

I just used in and it threw an error for timing out

Auger•2/3/23, 1:00 AM

Nah, you're awaiting, which is what you should be doing

Auger•2/3/23, 1:00 AM

Are you hitting the console line?

AAuger Nah, you're awaiting, which is what you should be doing

Japemaster BradOP•2/3/23, 1:01 AM

Cuts out as I step onto it

Auger•2/3/23, 1:01 AM

So, it isn't making it past line 17?

AAuger So, it isn't making it past line 17?

Japemaster BradOP•2/3/23, 1:01 AM

Auger•2/3/23, 1:02 AM

The timeout exception you saw, could indicate the issue. If this is a GET request, you can try doing the same request from either a browser or postman to look into it

Auger•2/3/23, 1:02 AM

You may want to ensure the API is reachable at all

Auger•2/3/23, 1:03 AM

Try pasting the url you're trying to hit in your code, inside your browser

AAuger The timeout exception you saw, could indicate the issue. If this is a GET reques...

Japemaster BradOP•2/3/23, 1:03 AM

I actually built it in Postman first, then transferred all the data into VS, that's what confused me at first, it's almost like it's getting the response, ticking the box and cutting out

Auger•2/3/23, 1:03 AM

You'll likely see a 401 since you won't have a bearer, but that at least means the API is reachable

JJapemaster Brad I actually built it in Postman first, then transferred all the data into VS, tha...

Auger•2/3/23, 1:04 AM

To me, it sounds like the async method isn't resolving and hanging until the default http timeout is hit

Auger•2/3/23, 1:04 AM

So, I'd double check your

Auger•2/3/23, 1:05 AM

Especially since you're concatenating strings to make it

AAuger Especially since you're concatenating strings to make it

Japemaster BradOP•2/3/23, 1:07 AM

Combed over it, and nothing

Auger•2/3/23, 1:08 AM

Try setting a breakpoint on line 17, and take that URL and post it into your browser. I would expect a 401, and it not to time out

AAuger Try setting a breakpoint on line 17, and take that URL and post it into your bro...

Japemaster BradOP•2/3/23, 1:10 AM

JJapemaster Brad Click to see attachment

Auger•2/3/23, 1:45 AM

Yeah, so that's what I'd expect. So does it resolve in postman when you pass the bearer token?

AAuger Yeah, so that's what I'd expect. So does it resolve in postman when you pass the...

Japemaster BradOP•2/3/23, 1:45 AM

I get a 200 with JSON if that's what you mean

Auger•2/3/23, 1:46 AM

Hmm.. You might need more headers in your httpclient in C#

AAuger Hmm.. You might need more headers in your httpclient in C#

Japemaster BradOP•2/3/23, 1:47 AM

Honestly that could be it, I'll have a look at the documentation again and have a read. Will get back to you with findings, thanks for your help!

TeBeCo•2/3/23, 5:11 AM

if that API returns a 200 on that error it's a fucking bug

TeBeCo•2/3/23, 5:12 AM

I would expect it to be a 401

TeBeCo•2/3/23, 5:12 AM

did you do as suggested ?

TeBeCo•2/3/23, 5:12 AM

replace with

TeBeCo•2/3/23, 5:12 AM

and console log the

TeBeCo•2/3/23, 5:12 AM

and wrap it all witha try/catch

TeBeCo•2/3/23, 5:13 AM

and either

TeBeCo•2/3/23, 5:13 AM

the fact that it throw make it not possible to be a 200 and throw there

TeBeCo•2/3/23, 5:14 AM

the httpclient throw

if the DNS is not resolve (target server down)
if status code is not success "greater than 399"

TeBeCo•2/3/23, 5:14 AM

in your case having a body mean something reply

TeBeCo•2/3/23, 5:14 AM

so it mean status code

TeBeCo•2/3/23, 5:14 AM

log it / show us both your new code and the real log you added

TeBeCo•2/3/23, 5:15 AM

try
{
var resposne = ... getasync()
log(response.statuscode);
}
catch(exception ex)
{
log(ex)
}

TeBeCo•2/3/23, 5:15 AM

stuff like that

demidev_mb•2/3/23, 5:33 AM

Disable just my code in debug settings, and enable all exception types so visual studio cache everything

retry and see what's actually happening.

TeBeCo•2/3/23, 5:45 AM

please do the try/catch and logs anyway, the enable exception is just a bonus, you;ll still want your app to have proper code

TeBeCo•2/3/23, 5:46 AM

you'll still need to try/catch and possibly to log

TeBeCo•2/3/23, 5:46 AM

enabling all exception might help for that one time error but won't fix your code

TeBeCo•2/3/23, 5:46 AM

you're doing httpclient + async code => you absolutly need to try/catch at some point

TeBeCo•2/3/23, 5:47 AM

that's why the program stop working

demidev_mb•2/3/23, 5:50 AM

Yeah, try cache is required anyway

TeBeCo•2/3/23, 5:51 AM

I wish "jsut my code + enable exception" would be better known
but also how to reset that after lmao

Henkypenky•2/3/23, 5:52 AM

is this the thing where the console closes after finishing everything

Henkypenky•2/3/23, 5:52 AM

there is a setting for that

TeBeCo•2/3/23, 5:52 AM

it's the thing that force the debigger to throw at the line the exception did throw