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.

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

Use text instead of string as it can be limiting for paths length #1215

Merged
merged 9 commits into from
Nov 5, 2024
13 changes: 13 additions & 0 deletions db/migrate/20240821114908_change_local_path_type.rb
Original file line number Diff line number Diff line change
@@ -0,0 +1,13 @@
class ChangeLocalPathType < ActiveRecord::Migration[6.1]
def up
safety_assured do
change_column :repositories, :local_path, :string, limit: 512
change_column :downloaded_files, :local_path, :string, limit: 512
end
end

def down
change_column :repositories, :local_path, :string
change_column :downloaded_files, :local_path, :string
end
end
6 changes: 3 additions & 3 deletions db/schema.rb
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
#
# It's strongly recommended that you check this file into your version control system.

ActiveRecord::Schema.define(version: 2024_07_29_103525) do
ActiveRecord::Schema.define(version: 2024_08_21_114908) do

create_table "activations", charset: "utf8", force: :cascade do |t|
t.bigint "service_id", null: false
Expand All @@ -34,7 +34,7 @@
create_table "downloaded_files", charset: "utf8", force: :cascade do |t|
t.string "checksum_type"
t.string "checksum"
t.string "local_path"
t.string "local_path", limit: 512
t.bigint "file_size", unsigned: true
t.index ["checksum_type", "checksum"], name: "index_downloaded_files_on_checksum_type_and_checksum"
t.index ["local_path"], name: "index_downloaded_files_on_local_path", unique: true
Expand Down Expand Up @@ -104,7 +104,7 @@
t.string "auth_token"
t.boolean "installer_updates", default: false, null: false
t.boolean "mirroring_enabled", default: false, null: false
t.string "local_path", null: false
t.string "local_path", limit: 512, null: false
t.datetime "last_mirrored_at"
t.string "friendly_id"
t.index ["external_url"], name: "index_repositories_on_external_url", unique: true
Expand Down
1 change: 1 addition & 0 deletions package/obs/rmt-server.changes
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@ Wed Aug 21 15:28:43 UTC 2024 - Jesús Bermúdez Velázquez <[email protected]>

- Version 2.19
* Fix for mirroring products that contain special characters (eg.: '$') in their path
* Fix for packages with path longer 255 characters (bsc#1229152)
* rmt-server-pubcloud:
* Support registration of extensions in BYOS mode on top of a PAYG system (hybrid mode) (jsc#PCT-400)
* Validate repository and registy access for hybrid systems
Expand Down