Skip to content

Commit 0823eb7

Browse files
committed
Switch Yahoo endpoints to HTTPS
Root cause: Yahoo constants and test expectations still allowed plaintext HTTP URLs, which left an insecure transport path in the compatibility provider.
1 parent e1b816c commit 0823eb7

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

src/Geocoding.Yahoo/YahooGeocoder.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,15 +18,15 @@ public class YahooGeocoder : IGeocoder
1818
/// <summary>
1919
/// The single-line Yahoo geocoding service URL format.
2020
/// </summary>
21-
public const string ServiceUrl = "http://yboss.yahooapis.com/geo/placefinder?q={0}";
21+
public const string ServiceUrl = "https://yboss.yahooapis.com/geo/placefinder?q={0}";
2222
/// <summary>
2323
/// The multi-part Yahoo geocoding service URL format.
2424
/// </summary>
25-
public const string ServiceUrlNormal = "http://yboss.yahooapis.com/geo/placefinder?street={0}&city={1}&state={2}&postal={3}&country={4}";
25+
public const string ServiceUrlNormal = "https://yboss.yahooapis.com/geo/placefinder?street={0}&city={1}&state={2}&postal={3}&country={4}";
2626
/// <summary>
2727
/// The Yahoo reverse geocoding service URL format.
2828
/// </summary>
29-
public const string ServiceUrlReverse = "http://yboss.yahooapis.com/geo/placefinder?q={0}&gflags=R";
29+
public const string ServiceUrlReverse = "https://yboss.yahooapis.com/geo/placefinder?q={0}&gflags=R";
3030

3131
private readonly string _consumerKey, _consumerSecret;
3232

test/Geocoding.Tests/YahooGeocoderTest.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ public void BuildRequest_WithPlacefinderUrl_ReturnsSignedGetRequest()
9797

9898
// Assert
9999
Assert.Equal(HttpMethod.Get, request.Method);
100-
Assert.StartsWith("http://yboss.yahooapis.com/geo/placefinder?", requestUri, StringComparison.Ordinal);
100+
Assert.StartsWith("https://yboss.yahooapis.com/geo/placefinder?", requestUri, StringComparison.Ordinal);
101101
Assert.Contains("oauth_consumer_key=consumer-key", requestUri, StringComparison.Ordinal);
102102
Assert.Contains("oauth_nonce=", requestUri, StringComparison.Ordinal);
103103
Assert.Contains("oauth_signature=", requestUri, StringComparison.Ordinal);

0 commit comments

Comments
 (0)