Skip to content
  • Categories
  • Recent
  • Popular
Skins
  • Light
  • Cerulean
  • Cosmo
  • Flatly
  • Journal
  • Litera
  • Lumen
  • Lux
  • Materia
  • Minty
  • Morph
  • Pulse
  • Sandstone
  • Simplex
  • Sketchy
  • Spacelab
  • United
  • Yeti
  • Zephyr
  • Dark
  • Cyborg
  • Darkly
  • Quartz
  • Slate
  • Solar
  • Superhero
  • Vapor

  • Default (No Skin)
  • No Skin
Collapse

NodeBB-ActivityPub Bridge Test Instance

  1. Home
  2. Categories
  3. Uncategorized
  4. okay, I fixed two bugs and posts with attachments are still invisible to mastodont, so I'm redefining success as reduced bandwidth requirements.

okay, I fixed two bugs and posts with attachments are still invisible to mastodont, so I'm redefining success as reduced bandwidth requirements.

Scheduled Pinned Locked Moved Uncategorized
8 Posts 2 Posters 16 Views
  • Oldest to Newest
  • Newest to Oldest
  • Most Votes
Reply
  • Reply as topic
Log in to reply
This topic has been deleted. Only users with topic management privileges can see it.
  • tedu@honk.tedunangst.comT This user is from outside of this forum
    tedu@honk.tedunangst.comT This user is from outside of this forum
    tedu@honk.tedunangst.com
    wrote last edited by
    #1

    okay, I fixed two bugs and posts with attachments are still invisible to mastodont, so I'm redefining success as reduced bandwidth requirements. Mission accomplished.

    tedu@honk.tedunangst.comT 1 Reply Last reply
    0
    • tedu@honk.tedunangst.comT tedu@honk.tedunangst.com

      okay, I fixed two bugs and posts with attachments are still invisible to mastodont, so I'm redefining success as reduced bandwidth requirements. Mission accomplished.

      tedu@honk.tedunangst.comT This user is from outside of this forum
      tedu@honk.tedunangst.comT This user is from outside of this forum
      tedu@honk.tedunangst.com
      wrote last edited by
      #2

      If I've read the code in app/lib/activitypub/parser/media_attachment_parser.rb right, mastodon only supports a single string value for the url property. No arrays, no Link types. Curiously, this seems to prevent the entire activity from appearing, not just the image.

      @trwnh you're my AP compliance officer contact.

      trwnh@mastodon.socialT 1 Reply Last reply
      0
      • tedu@honk.tedunangst.comT tedu@honk.tedunangst.com

        If I've read the code in app/lib/activitypub/parser/media_attachment_parser.rb right, mastodon only supports a single string value for the url property. No arrays, no Link types. Curiously, this seems to prevent the entire activity from appearing, not just the image.

        @trwnh you're my AP compliance officer contact.

        trwnh@mastodon.socialT This user is from outside of this forum
        trwnh@mastodon.socialT This user is from outside of this forum
        trwnh@mastodon.social
        wrote last edited by
        #3

        @tedu that’s weird, i guess i can file a mastodon bug on your behalf if you provide me a sample payload

        trwnh@mastodon.socialT tedu@honk.tedunangst.comT 2 Replies Last reply
        0
        • trwnh@mastodon.socialT trwnh@mastodon.social

          @tedu that’s weird, i guess i can file a mastodon bug on your behalf if you provide me a sample payload

          trwnh@mastodon.socialT This user is from outside of this forum
          trwnh@mastodon.socialT This user is from outside of this forum
          trwnh@mastodon.social
          wrote last edited by
          #4

          @tedu looking at https://honk.tedunangst.com/u/tedu/h/d21rx2s3jrsbNP2Nbc my only theoretical nitpicks are that it should be an Image attachment instead of a Document, and also maybe it would be nice if the `context` was something resolvable

          that probably doesn’t solve this specific problem, but in the spirit of correctness i thought i’d mention those 😛

          trwnh@mastodon.socialT 1 Reply Last reply
          0
          • trwnh@mastodon.socialT trwnh@mastodon.social

            @tedu looking at https://honk.tedunangst.com/u/tedu/h/d21rx2s3jrsbNP2Nbc my only theoretical nitpicks are that it should be an Image attachment instead of a Document, and also maybe it would be nice if the `context` was something resolvable

            that probably doesn’t solve this specific problem, but in the spirit of correctness i thought i’d mention those 😛

            trwnh@mastodon.socialT This user is from outside of this forum
            trwnh@mastodon.socialT This user is from outside of this forum
            trwnh@mastodon.social
            wrote last edited by
            #5

            @tedu oh, also arrays with single values should be coerced to just those values unless the property is defined as @/type: @/list or @/type: @/set

            so the cc: [followers] should be just cc: followers

            (yes, mastodon doesn’t comply with this in what it produces)

            trwnh@mastodon.socialT 1 Reply Last reply
            0
            • trwnh@mastodon.socialT trwnh@mastodon.social

              @tedu oh, also arrays with single values should be coerced to just those values unless the property is defined as @/type: @/list or @/type: @/set

              so the cc: [followers] should be just cc: followers

              (yes, mastodon doesn’t comply with this in what it produces)

              trwnh@mastodon.socialT This user is from outside of this forum
              trwnh@mastodon.socialT This user is from outside of this forum
              trwnh@mastodon.social
              wrote last edited by
              #6

              @tedu issue filed https://github.com/mastodon/mastodon/issues/34979

              1 Reply Last reply
              0
              • trwnh@mastodon.socialT trwnh@mastodon.social

                @tedu that’s weird, i guess i can file a mastodon bug on your behalf if you provide me a sample payload

                tedu@honk.tedunangst.comT This user is from outside of this forum
                tedu@honk.tedunangst.comT This user is from outside of this forum
                tedu@honk.tedunangst.com
                wrote last edited by
                #7

                @trwnh thanks. I'm not using github. you can probably guess where I copied Document and cc[] from. 🙂

                trwnh@mastodon.socialT 1 Reply Last reply
                0
                • tedu@honk.tedunangst.comT tedu@honk.tedunangst.com

                  @trwnh thanks. I'm not using github. you can probably guess where I copied Document and cc[] from. 🙂

                  trwnh@mastodon.socialT This user is from outside of this forum
                  trwnh@mastodon.socialT This user is from outside of this forum
                  trwnh@mastodon.social
                  wrote last edited by
                  #8

                  @tedu a fix was just merged: https://github.com/mastodon/mastodon/pull/34996

                  1 Reply Last reply
                  0
                  Reply
                  • Reply as topic
                  Log in to reply
                  • Oldest to Newest
                  • Newest to Oldest
                  • Most Votes


                  • Login

                  • Login or register to search.
                  Powered by NodeBB Contributors
                  • First post
                    Last post
                  0
                  • Categories
                  • Recent
                  • Popular