pylon-api 1.1.0 → 1.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6d98b4389f4182e2f3d62261c2a60d9f4a325c18a5376c09668cc17e1594219c
-  data.tar.gz: ea1c690a74e4e8d2890098211d58909ae551af8deecaf757d3463e5de37f3cdc
+  metadata.gz: 4f54f7ed75e823866e06256d1a598bb3158e686131156932b7aad3d9a90da9c7
+  data.tar.gz: fe68e2c1be8128acadd4179b2baa2751fd9223fdbe1a5c66cb35d2d41c13fae0
 SHA512:
-  metadata.gz: b3a1dcd703454b3efc5818292f01a00366a50ddfa5f06d6dbbdbfc865907b907b05ea35d0ca591e0eb93d5ffbec26e4ad90b85e47379aa029f61346b766e81f1
-  data.tar.gz: aa703a9d4f20da2243ce49b31627548afd719835a3a591fab2ffafd5480ab84cbce02b3c88a54b25979a8138641b584c9e32054ae673f548ca4936daa2531e53
+  metadata.gz: 6282c28251d01ed1f7c196fbb421c3e895d37681da8e71e5941ed74db3ff88aff39e8e9c8604fe899d555e61aec5bc7b8824fcdf30e6e789c3b74086d25ae7ee
+  data.tar.gz: bf00a26818a96154c91a2a1f281d219372f0fd97b8b0075ddbb70fcebaa675c28b31e55a10bf218303fcc4e2b0ad15229a30950e2f2a5e41cd174995940c0a2a

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [1.1.1] - 2025-04-18
+
+### Fixed
+- Fixed URL-based file attachments by ensuring they use proper multipart form encoding
+
+## [1.1.0] - 2025-04-17
+
+### Added
+- Support for URL-based file attachments
+- Enhanced attachment handling for various file types
+- Added MIME type detection for file uploads
+
 ## [1.0.0] - 2024-03-21
 
 ### Changed

data/lib/pylon/client.rb

@@ -7,7 +7,22 @@ module Pylon
   #   client = Pylon::Client.new(api_key: 'your_api_key')
   #   issues = client.list_issues(start_time: '2024-03-01T00:00:00Z', end_time: '2024-03-31T23:59:59Z')
   #   issues.each { |issue| puts issue.title }
+  # rubocop:disable Metrics/ClassLength
   class Client
+    # Common MIME types for file uploads
+    MIME_TYPES = {
+      ".jpg" => "image/jpeg",
+      ".jpeg" => "image/jpeg",
+      ".png" => "image/png",
+      ".gif" => "image/gif",
+      ".pdf" => "application/pdf",
+      ".doc" => "application/msword",
+      ".docx" => "application/vnd.openxmlformats-officedocument.wordprocessingml.document",
+      ".xls" => "application/vnd.ms-excel",
+      ".xlsx" => "application/vnd.openxmlformats-officedocument.spreadsheetml.sheet",
+      ".csv" => "text/csv",
+      ".txt" => "text/plain"
+    }
     # Base URL for the Pylon API
     BASE_URL = "https://api.usepylon.com"
 
@@ -45,10 +60,36 @@ module Pylon
 
     # Create a new attachment
     #
-    # @param file [String] The file content to upload
+    # @param file [File, String, nil] The file to upload (File object or file content)
+    # @param description [String, nil] A text description of the file
+    # @param file_url [String, nil] A URL to fetch the file from if file is not provided
     # @return [Models::Attachment] Created attachment object
-    def create_attachment(file)
-      post("/attachments", body: { file: file }, model_class: Models::Attachment)
+    # @raise [ArgumentError] If neither file nor file_url is provided
+    def create_attachment(file = nil, description: nil, file_url: nil)
+      if file.nil? && file_url.nil?
+        raise ArgumentError, "Either file or file_url must be provided"
+      end
+
+      params = {}
+      params[:description] = description if description
+      params[:file_url] = file_url if file_url
+
+      if file
+        if file.is_a?(::File) || (file.respond_to?(:path) && file.respond_to?(:read))
+          # For File objects or IO-like objects
+          params[:file] = Faraday::UploadIO.new(file, mime_type_for_file(file), File.basename(file.path))
+        else
+          # For raw content as string, create a temp file
+          temp_file = Tempfile.new(["upload", ".bin"])
+          temp_file.binmode
+          temp_file.write(file)
+          temp_file.rewind
+          params[:file] = Faraday::UploadIO.new(temp_file, "application/octet-stream")
+        end
+      end
+      
+      # Always use multipart for attachments, whether file or file_url
+      post_multipart("/attachments", params, model_class: Models::Attachment)
     end
 
     # Get details for a specific attachment
@@ -324,6 +365,17 @@ module Pylon
       end
     end
 
+    # Determine MIME type for file
+    #
+    # @param file [File] The file to determine MIME type for
+    # @return [String] The MIME type
+    def mime_type_for_file(file)
+      return "application/octet-stream" unless file.respond_to?(:path)
+
+      ext = File.extname(file.path).downcase
+      MIME_TYPES[ext] || "application/octet-stream"
+    end
+
     # Handle API response and raise appropriate errors
     #
     # @param response [Faraday::Response] The API response
@@ -427,6 +479,34 @@ module Pylon
       handle_response(connection.post(path, body.to_json), model_class, collection)
     end
 
+    # Make a multipart POST request for file uploads
+    #
+    # @param path [String] The API endpoint path
+    # @param params [Hash] Request parameters including file uploads
+    # @param model_class [Class] The model class to use for wrapping the response
+    # @param collection [Boolean] Whether the response is a collection of items
+    # @return [Models::Base, Models::Collection, Array] Model, Collection, or response data array
+    def post_multipart(path, params = {}, model_class: nil, collection: false)
+      # Create a connection without the default JSON content type for multipart uploads
+      multipart_conn = Faraday.new(@base_url) do |f|
+        f.request :multipart
+        # Only force multipart for file_url uploads, omit url_encoded middleware
+        f.response :json, content_type: /\bjson$/
+        f.response :logger if @debug
+        f.adapter Faraday.default_adapter
+        f.headers["Authorization"] = "Bearer #{api_key}"
+        f.headers["Accept"] = "application/json"
+      end
+      
+      # If we have a file_url but no actual file, create a dummy file part to force multipart encoding
+      if params[:file_url] && !params[:file]
+        # Create an empty file part to force multipart encoding
+        params[:_dummy] = Faraday::FilePart.new(StringIO.new(""), "application/octet-stream", "dummy")
+      end
+
+      handle_response(multipart_conn.post(path, params), model_class, collection)
+    end
+
     # Make a PATCH request
     #
     # @param path [String] The API endpoint path

data/lib/pylon/version.rb

@@ -6,7 +6,7 @@ module Pylon
   # Minor version for new features
   MINOR = 1
   # Patch version for bug fixes
-  PATCH = 0
+  PATCH = 1
   # Pre-release version (optional)
   PRE = nil
 

data/lib/pylon.rb

@@ -3,6 +3,7 @@
 require "faraday"
 require "faraday/multipart"
 require "json"
+require "tempfile"
 
 require_relative "pylon/version"
 require_relative "pylon/models/base"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pylon-api
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Ben Odom
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-17 00:00:00.000000000 Z
+date: 2025-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday