API לקבלת שערי מט"ח

dovid

בתכנות, תמיד תמיד יש תקלות ותמיהות.
אבל זה לא עוצר אותך מלהתקדם אם אתה לא ניגש לבעיה ככה:

אבל מקוד זה לא עובד

אם אתה רוצה להתקדם בחיים,
אתה צריך להבין שמה שלא עובד זה מרתק/מקדם לא פחות ממה שכן, ושווה להשקיע מחקר לראות מה בדיוק קורה שמה.

dovid

אוקי בא אספר לך בינתיים איך התקדמתי בבעיה.
לקחתי את הקוד שלך ופתחתי את הקונסול של הדפדפן והדבקתי (בתוספת שורה שמפעילה את הפונקציה).
קיבלתי שגיאה ארוכה של Mixed Content שאומרת שאני מנסה לגשת לhttp מתוך דף מאובטח של https.
אז התקדמתי, ושיניתי בקוד שלך לhttps והרצתי שוב, וקיבלתי שגיאה ארוכה לא פחות של CORS policy, כלומר אתר תחומים לא מורשה לגשת לבנק ישראל.
אז הלכתי לאתר בנק ישראל ומשמה פתחתי את הקונסול והרצתי, ושמה קיבלתי שגיאה שקשורה לJSON - התוכן מתחיל בתו לא צפוי שהוא <.
הסתכלתי בNetwork לראות איך נראית התשובה, והנה היא XML! בטח הרי זה כתוב מפורש בכתובת currency.xml רק שלא שמתי לב.
טוב אז אני יודע לענות לך בפורום שהבעיה היא שזה לא JSON.

dovid

שו"ר ש@yossiz כבר ענה על השאלה פה:
https://tchumim.com/topic/9348

ש.ב.ח.

@dovid אמר בAPI לקבלת שערי מט"ח:

אם אתה רוצה להתקדם בחיים,
אתה צריך להבין שמה שלא עובד זה מרתק/מקדם לא פחות ממה שכן, ושווה להשקיע מחקר לראות מה בדיוק קורה שמה.

בשביל זה גם ניסיתי הכל וגם אני נתקלתי בcros...
(חשוב להדגיש שבאתר הבנק הכתובת מופיעה HTTP ולא HTTPS, לא שזה ראיה גדולה...)
וכבר הבנתי שהבנק לא מרשה...

גם ידעתי שככל הנראה בסוף תהיה הבעיה של הJSON רק חיכיתי וציפיתי לראות את השגיאה הזאת עולה (הבנתי שגם לזה אצטרך למצוא פיתרון) אבל כל עוד היא לא עלתה... לא היה לי מה להמשיך.

בכל מקרה לא הבנתי מה כתב @yossiz ב
@yossiz אמר בjs- שימוש ב fetch לקובץ xml:

@ב-ל צודק, כנראה שזה לא מיועד לשימוש בצד לקוח (או שהם עדיין לא שמעו על CORS...)

האם בצד שרת זה כן יעבוד?

yossiz

@ש-ב-ח מדיניית ה-CORS מואכפת רק בדפדפן - לא בשרת.

ש.ב.ח.

@yossiz אמר בAPI לקבלת שערי מט"ח:

@ש-ב-ח מדיניית ה-CORS מואכפת רק בדפדפן - לא בשרת.

(יש לי עוד הרבה מה ללמוד, זה כנראה יקח זמן, אתם (גם @dovid ) תצטרכו לעזור לי להשלים את הפער... )
תודה על הכל!

ש.ב.ח.

@yossiz
גם מצד שרת הוא לא נותן
הוא מחזיר 404

האם ידוע API אחר לבדיקת שערים?

chv

@ש-ב-ח זה הכי טוב - חינמי - מהיר
עדכני (אאלט כל חצי שעה זה מתעדכן)
https://www.currencyconverterapi.com/

מחזיר JSON. עובד גם בצד לקוח כמובן (הרשאת CORS)

yossiz

@ש-ב-ח כנראה שעשית משהו לא נכון.
הרצתי משרת:

curl https://www.boi.org.il/currency.xml?curr=01

וקיבלתי במענה:

<?xml version="1.0" encoding="utf-8" standalone="yes"?>
<CURRENCIES>
  <LAST_UPDATE>2021-03-15</LAST_UPDATE>
  <CURRENCY>
    <NAME>Dollar</NAME>
    <UNIT>1</UNIT>
    <CURRENCYCODE>USD</CURRENCYCODE>
    <COUNTRY>USA</COUNTRY>
    <RATE>3.309</RATE>
    <CHANGE>-0.211</CHANGE>
  </CURRENCY>
</CURRENCIES>

dovid

@ש-ב-ח אמר בAPI לקבלת שערי מט"ח:

@yossiz
גם מצד שרת הוא לא נותן

המילים "לא נותן" "לא עובד", לא נסבלים.
תכתוב מה התוצאה, מה השגיאה מה הפלט.

ש.ב.ח.

@dovid אמר בAPI לקבלת שערי מט"ח:

המילים "לא נותן" "לא עובד", לא נסבלים.
תכתוב מה התוצאה, מה השגיאה מה הפלט.

@ש-ב-ח אמר בAPI לקבלת שערי מט"ח:

הוא מחזיר 404

dovid

@ש-ב-ח תצרף קוד ותציין בדיוק איך אתה מריץ אותו (שרת, מחשב, דפדפן).

ש.ב.ח.

@yossiz אמר בAPI לקבלת שערי מט"ח:

וקיבלתי במענה:

אכן ככל הנראה הבעיה היא בפונקציה בה אני משתמש
אני משתמש ב wp_remote_get

אבל אני לא יודע מאיפה להתחיל כדי לבדוק מה הבעיה
אני מנסה לחפש...

yossiz

@ש-ב-ח בדקת ב-https?

ש.ב.ח.

@dovid
כך:

$res = wp_remote_get('https://www.boi.org.il/currency.xml?curr=01');
print_r($res);

בשרת (מנסה להציג את התוצאה בדף)

@yossiz אמר בAPI לקבלת שערי מט"ח:

@ש-ב-ח בדקת ב-https?

בוודאי

ש.ב.ח.

למשל עם הפונקציה הפשוטה של PHP זה עובד חלק

simplexml_load_file('https://www.boi.org.il/currency.xml');

מה לא עלינו לא טוב לו עם wp_remote_get()?
(נכון, אני יכול להסתדר עם הפונקציה הזאת של PHP אבל אני סקרן נורא לדעת מה לא טוב בפונקציה של WP)

yossiz

@ש-ב-ח הנה התשובה

wp_remote_get('https://www.boi.org.il/currency.xml?curr=01', ['user-agent' => 'me']);

dovid

@yossiz אולי תענה גם לו
https://wordpress.stackexchange.com/q/341706
...

אני מייד חשבתי בכיוון כזה אבל simplexml_load_file קובע בלי שהתבקש user_agent?
וcurl גם?

yossiz

@dovid אני הבנתי שהם בפירוש חוסמים ה-user-agent של וורדפרס. (יש להם user-agent בברירת מחדל שזה משהו כמו Wordpress/your-version; http://your-site)

yossiz

@dovid אמר בAPI לקבלת שערי מט"ח:

@yossiz אולי תענה גם לו
https://wordpress.stackexchange.com/q/341706

הוא מגדיר user-agent ועדיין לא עובד לו

תחומים - פורום חרדי מקצועי

API לקבלת שערי מט"ח