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.

Sign up for GitHub

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

Jump to bottom

amqp1 issue #27

Closed
paramite opened this issue Jun 5, 2023 · 2 comments
Closed

amqp1 issue #27

paramite opened this issue Jun 5, 2023 · 2 comments

Comments

@paramite
Copy link
Member

paramite commented Jun 5, 2023

Building collectd-sensubility with qpid-proton-c version higher than 0.34.0 makes sensubility to fail during sending second message with:

panic: cannot marshal string: overflow: not enough space to encode

goroutine 8 [running]:
github.com/apache/qpid-proton/go/pkg/amqp.marshal({0x68c3e0?, 0xc00012a410?}, 0x0?)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/amqp/marshal.go:293 +0xa47
github.com/apache/qpid-proton/go/pkg/amqp.putData({0x68c3e0, 0xc00012a410}, 0x44c0e5?)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/amqp/message.go:508 +0x50
github.com/apache/qpid-proton/go/pkg/amqp.(*message).put(0xc000154000, 0x200000003?)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/amqp/message.go:560 +0x2b8
github.com/apache/qpid-proton/go/pkg/amqp.(*MessageCodec).Encode(0x0?, {0x758e20?, 0xc000154000?}, {0x0, 0x0, 0x0})
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/amqp/message.go:380 +0x9f
github.com/apache/qpid-proton/go/pkg/electron.(*sender).send(0xc0002e0000, 0xc00011a630)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/electron/sender.go:197 +0x12b
github.com/apache/qpid-proton/go/pkg/electron.(*sender).trySend(0xc0002e0000)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/electron/sender.go:187 +0x25
github.com/apache/qpid-proton/go/pkg/electron.(*sender).startSend(...)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/electron/sender.go:179
github.com/apache/qpid-proton/go/pkg/electron.(*sender).SendAsyncTimeout.func1()
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/electron/sender.go:230 +0xc5
github.com/apache/qpid-proton/go/pkg/proton.(*Engine).Run(0xc0000f2360)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/proton/engine.go:376 +0x14f
github.com/apache/qpid-proton/go/pkg/electron.(*connection).run(0xc0001880f0)
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/electron/connection.go:241 +0x45
created by github.com/apache/qpid-proton/go/pkg/electron.NewConnection
/root/go/pkg/mod/github.com/apache/[email protected]/go/pkg/electron/connection.go:224 +0x570
@paramite
Copy link
Member Author

paramite commented Jun 5, 2023

We need to either pin on 0.34.0 or figure out what's the issue with the current amqp1 connector code.

paramite added a commit to centos-opstools/golang-github-infrawatch-apputils that referenced this issue Jun 14, 2023
@paramite
Pin on working qpid-proton
2086442 
There has been introduced change or bug in qpid-proton, which makes
connector malfunctioning.

More info: infrawatch/apputils#27
@paramite paramite mentioned this issue Jun 14, 2023
Merged
@paramite
Copy link
Member Author

paramite commented May 3, 2024

Fixed in PR #29.

@paramite paramite closed this as completed May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@paramite