Skip to content

Commit 3c9f480

Browse files
author
Dan Gidman
committed
SF-4342 adding a sanity check in for status code
1 parent d7c8704 commit 3c9f480

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

Src/StackifyLib/Models/EnvironmentDetail.cs

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -113,15 +113,19 @@ public static string GetEC2InstanceId()
113113
request.Timeout = 5000;
114114
using (var response = (HttpWebResponse) request.GetResponse())
115115
{
116-
var encoding = Encoding.GetEncoding(response.CharacterSet);
117-
using (var responseStream = response.GetResponseStream())
116+
if ((int) response.StatusCode >= 200 && (int) response.StatusCode < 300)
118117
{
119-
using (var reader = new StreamReader(responseStream, encoding))
118+
var encoding = Encoding.GetEncoding(response.CharacterSet);
119+
using (var responseStream = response.GetResponseStream())
120120
{
121-
var id = reader.ReadToEnd();
122-
return string.IsNullOrWhiteSpace(id) ? null : id;
121+
using (var reader = new StreamReader(responseStream, encoding))
122+
{
123+
var id = reader.ReadToEnd();
124+
return string.IsNullOrWhiteSpace(id) ? null : id;
125+
}
123126
}
124127
}
128+
return null;
125129
}
126130
}
127131
catch // if not in aws this will timeout

0 commit comments

Comments
 (0)