Merge pull request #131 from lacostej/fix/unicode

Replace non unicode characters to avoid crashes
tj · Jan 27, 2025 · 16bc707 · 16bc707
2 parents 2b36467 + 93723db
commit 16bc707
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/lib/terminal-table/cell.rb b/lib/terminal-table/cell.rb
@@ -48,7 +48,7 @@ def align(val, position, length)
         val.public_send(positions[position], length)
       end
       def lines
-        @value.to_s.split(/\n/)
+        @value.to_s.encode("utf-8", invalid: :replace).split(/\n/)
       end
 
       ##

diff --git a/spec/table_spec.rb b/spec/table_spec.rb
@@ -120,6 +120,18 @@ module Terminal
       EOF
     end
 
+    it "should replace illegal unicode characters" do
+      @table.headings = ["\xAE"]
+      @table << ['a']
+      @table.render.should eq <<-EOF.deindent
+        +---+
+        | � |
+        +---+
+        | a |
+        +---+
+      EOF
+    end
+
     it "should render styles properly" do
       @table.headings = ['Char', 'Num']
       @table.style = {