js-kompo-utils 1.0.35 → 1.0.37

package/components/VlInternationalPhoneInput.vue

@@ -181,6 +181,14 @@ export default {
             return `${baseClass} ${errorClass}`
         },
         
+        // An explicit country() is a hard setting. Otherwise the IP lookup runs when it is turned
+        // on — its result wins over defaultCountry — or when there is no default to start from.
+        shouldDetectByIp() {
+            if (this.$_config('country')) return false
+
+            return !!this.$_config('detectCountryByIp') || !this.selectedCountryCode
+        },
+
         // Current country info
         currentCountry() {
             const country = this.countries.find(c => c.code === this.selectedCountryCode)
@@ -289,14 +297,15 @@ export default {
     mounted() {
         // Close dropdown when clicking outside
         document.addEventListener('click', this.handleClickOutside)
-        
-        // Set initial value
-        // this.component.value = this.phoneValue
-        
-        // Auto-detect country if no explicit country is set and no phone value exists
+
+        // `component` is not guaranteed to be bound when data() runs — see phoneValue's `?.` — and
+        // a country read as undefined there would send us asking an IP service we don't need.
+        this.selectedCountryCode = this.selectedCountryCode || this.$_config('country') || this.$_config('defaultCountry')
+
+        // A stored number always decides its own country; an empty field may ask the IP.
         if (this.phoneValue) {
             this.autoDetectCountryFromPhone(this.phoneValue)
-        } else if (!this.selectedCountryCode) {
+        } else if (this.shouldDetectByIp) {
             this.detectCountryByLocation()
         }
     },
@@ -672,40 +681,25 @@ export default {
             }
         },
         
+        // Free, keyless, https + CORS. http:// providers are mixed content on an https page and
+        // never resolve, and a provider answering without a code (ipapi.co rate-limits often) has
+        // to fall through to the next one instead of ending the chain.
         async getCountryFromIP() {
-            // Try ipapi.co first (free, reliable)
-            try {
-                const response = await fetch('https://ipapi.co/json/', {
-                    timeout: 5000
-                })
-                const data = await response.json()
-                return data.country_code
-            } catch (error) {
-                console.warn('ipapi.co failed:', error)
-            }
-            
-            // Fallback to ip-api.com
-            try {
-                const response = await fetch('http://ip-api.com/json/', {
-                    timeout: 5000
-                })
-                const data = await response.json()
-                return data.countryCode
-            } catch (error) {
-                console.warn('ip-api.com failed:', error)
-            }
-            
-            // Fallback to ipinfo.io
-            try {
-                const response = await fetch('https://ipinfo.io/json', {
-                    timeout: 5000
-                })
-                const data = await response.json()
-                return data.country
-            } catch (error) {
-                console.warn('ipinfo.io failed:', error)
+            const providers = [
+                ['https://get.geojs.io/v1/ip/country.json', data => data.country],
+                ['https://ipwho.is/', data => data.success && data.country_code],
+                ['https://ipapi.co/json/', data => data.country_code],
+            ]
+
+            for (const [url, readCountry] of providers) {
+                try {
+                    const code = readCountry(await (await fetch(url)).json())
+                    if (code) return code
+                } catch (error) {
+                    console.warn(`${url} failed:`, error)
+                }
             }
-            
+
             return null
         },
         

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "js-kompo-utils",
-  "version": "1.0.35",
+  "version": "1.0.37",
   "description": "",
   "main": "app.js",
   "scripts": {