FEATURE: support picture api

config/settings.yml

@@ -24,6 +24,9 @@ mozilla_iam:
   mozilla_iam_person_v2_api_aud:
     default: "api.sso.mozilla.com"
     shadowed_by_global: true
+  mozilla_iam_picture_api_url:
+    default: "https://picture.api.sso.mozilla.com"
+    shadowed_by_global: true
   dinopark_url:
     default: "https://people.mozilla.org/"
     shadowed_by_global: true

lib/mozilla_iam/api/person_v2.rb

@@ -76,7 +76,7 @@ def process_full_name
         def process_picture
           url = process :picture
           if !url.blank? && url.starts_with?("/")
-            SiteSetting.dinopark_url.chomp("/") + url
+            SiteSetting.mozilla_iam_picture_api_url + url
           else
             url
           end

plugin.rb

@@ -1,6 +1,6 @@
 # name: mozilla-iam
 # about: A plugin to integrate Discourse with Mozilla's Identity and Access Management (IAM) system
-# version: 1.3.1
+# version: 1.4.0
 # authors: Leo McArdle
 # url: https://github.com/mozilla/discourse-mozilla-iam
 

spec/components/mozilla_iam/api/person_v2_spec.rb

@@ -171,9 +171,10 @@ def profile_with(attributes, value=nil, metadata={})
     describe "#picture" do
       context "with relative url" do
         let(:profile) { profile_with(:picture, "/avatar.png") }
+        before { SiteSetting.mozilla_iam_picture_api_url = "https://picture.api" }
 
         it "returns absolute url" do
-          expect(profile.public_send(:picture)).to eq "https://people.mozilla.org/avatar.png"
+          expect(profile.public_send(:picture)).to eq "https://picture.api/avatar.png"
         end
       end
     end