Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

【5.1】WebサイトURLの書式チェックが行われていません【システム基本設定＞WebサイトURL】 #3154

Closed
matsunaga-y opened this issue Feb 22, 2024 · 2 comments
Closed

【5.1】WebサイトURLの書式チェックが行われていません【システム基本設定＞WebサイトURL】 #3154

matsunaga-y opened this issue Feb 22, 2024 · 2 comments
Assignees
@HungDV2022
Labels
Todo 実行確定タスク

Comments

@matsunaga-y
Copy link

概要

【5.1】WebサイトURLの書式チェックが行われていません【システム基本設定＞WebサイトURL】

WebサイトURLにhttpsを消しても保存ができてしまいます。
WebサイトURLの書式チェックが行われていません

baserCMS version : 5.0.9-dev

TODO

備考
@ryuring
Copy link
Collaborator

ryuring commented Feb 23, 2024

@HungDV2022 こちら、スクリーンショットが違うようですが、【システム基本設定＞WebサイトURL】にURL形式のバリデーションの追加をお願いします。
スラッシュから始まるURLは許容せず、http、https のみ許容するようにお願いします。
SiteConfigs::validationKeyValue() で実装してください。

@ryuring ryuring added Bug バグ Todo 実行確定タスク and removed Bug バグ labels Feb 23, 2024
HungDV2022 pushed a commit to HungDV2022/basercms that referenced this issue Feb 26, 2024
@dovanhung
fix baserproject#3154 【5.1】WebサイトURLの書式チェックが行われていません【システム基本設定＞WebサイトURL】
c071154
ryuring pushed a commit that referenced this issue Feb 26, 2024
@HungDV2022 @dovanhung
fix #3154 【5.1】WebサイトURLの書式チェックが行われていません【システム基本設定＞WebサイトURL】 (#3157)
3f42aab 
Co-authored-by: Đỗ Văn Hùng <[email protected]>
@ryuring
Copy link
Collaborator

ryuring commented Feb 26, 2024

@matsunaga-y こちらのコミットで改善しました。
#3157

@ryuring ryuring closed this as completed Feb 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@HungDV2022 HungDV2022

Labels
Todo 実行確定タスク
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ryuring @HungDV2022 @matsunaga-y