너무 긴 `title` 설정으로 인해 백업이 생성되지 않습니다

기여 버그
1

X자리를 초과하는 title을 설정하면 백업이 생성되지 않습니다.

예외: /var/www/discourse/lib/discourse.rb:139:in 'exec': rm: cannot remove '/var/www/discourse/public/backups/default/redacted-xxx(...)xxx-2025-01-17-125027-v20250116024516.tar': File name too long

사실 사람들이 정말로 그렇게 합니다.

제안된 수정 사항

    def get_parameterized_title
      SiteSetting.title.parameterize.presence[..64] || "discourse"
    end
4개의 좋아요
2

그렇게 하면 안 되죠!

그분들의 주장이, 이번에도 그들에게 틀렸다고 말하지 말아야 한다는 것이구나 싶습니다.

2개의 좋아요
3

수정이 더 정교해질 필요가 있을 수 있습니다:

> "".parameterize.presence[..64]
NoMethodError: undefined method `[]' for nil

하지만 이 코드는 작동해야 합니다.

SiteSetting.title.parameterize[...64].presence || "discourse"

사이트 제목의 현재 최대 길이 제한에 대해 우리가 만족하는지 재고해 볼 필요가 있을 것 같습니다 @hugh

cc @gerhard

5개의 좋아요
4

다음 PR로 수정될 예정입니다:

5개의 좋아요
5

정보 주셔서 정말 감사합니다.

6

이 주제는 2일 후 자동으로 닫혔습니다. 새로운 답변은 더 이상 허용되지 않습니다.