NRE in SimpleWebSource.GetData()

[CodingGorilla]

If you have a update feed that does not specify a BaseUrl attribute on the feed element, then the UpdateManager.BaseUrl property is left null. When the SimpleWebSource.GetData() method is invoked, it does:

if (!baseUrl.EndsWith("/")) baseUrl += "/";

Which results in a NullReferenceException.

At a minimum there should be a String.IsNullOrEmpty() check on baseUrl, but it might be a good idea to make the UpdateManager.BaseUrl property public so it an be set by the client code.

[synhershko]

Thanks. Already fixed locally by adding a null check. I'm mid-work on some other issues and will push once done.

[synhershko]

Pushed now, thanks