SeAT - forum: Failed Jobs - UserRoleAddedListener - Page 1
suineg
12 Jan 2025 17:30
I think this might be happening because I don't have slack/discord/etc attached but we don't use any of that integration. Something I can do to fix this? Is it actually a problem or not? Log attached is the stack trace.
I'm running in docker.
Crypta Electrica
13 Jan 2025 00:36
This doesnt look like it comes from a lack of integrations set up.
What versions are you running of everything?
suineg
13 Jan 2025 00:36
v5
Crypta Electrica
13 Jan 2025 00:37
....
suineg
13 Jan 2025 00:37
if you're asking docker containers
Crypta Electrica
13 Jan 2025 00:37
Of the plugins and packages
suineg
13 Jan 2025 00:38
SEAT_PLUGINS=recursivetree/seat-rat,pyTonicis/seat-corp-mining-tax,recursivetree/seat-info,hermesdj/seat-calendar,cryptatech/seat-fitting,cryptatech/seat-srp,cryptatech/seat-prices-janice
Let me know if I can dump relevant logs and for sure not super worried just ensuring there is nothing I should be fixing here to ensure I have nothing wrong.
I have a separate question on the bootstrap CVE that I can sit on because I'm mitigating it and assume you're already tracking it and it's not a big issue that will get scooped up in a later commit
Crypta Electrica
13 Jan 2025 00:45
The bootstrap CVE does not impact us as we do not use that component
suineg
13 Jan 2025 00:46
got it, I'm a cybersec guy and not a hard coder either front or backend
Crypta Electrica
13 Jan 2025 00:47
I both like and dislike the CVE approach. I like to know about vulnerabilities in packages, but they typically (IMO) inflate their level of impact and leave no room in the eyes of compliance as to whether the CVE even matters.
(/me rants because I have to deal with 'CVE compliance' IRL and people managing them who do not understand the implications of statements like 'we must have 0 CVEs in our code')
suineg
13 Jan 2025 00:48
yeah very familiar with that (I work for Palo Alto Networks)
and my customer is the DoD where I spent my last 25 years
zero tolerance for any CVE 😄
Crypta Electrica
13 Jan 2025 00:51
My bad, I was looking at the wrong version of seat-connector. I forgot it was forked
yes it looks like it is because you do not have any drivers installed. Interesting side case.
Is there a reason why you have connector but no dirvers?
suineg
13 Jan 2025 00:52
got it, and that's core so not something I can disable
did I install it separately?
we don't hook into any connected chats
Crypta Electrica
13 Jan 2025 00:53
Yeah, thats not part of core.
suineg
13 Jan 2025 00:53
which plugin was that in, not seeing any named that is it the calendar?
Crypta Electrica
13 Jan 2025 00:54
It must have been pulled in by another plugin, easy way to check would be to ask composer
suineg
13 Jan 2025 00:54
yeah I see it there
Crypta Electrica
13 Jan 2025 00:54
so in your /opt/seat-docker dir run the following command
docker compose exec front composer depends warlof/seat-connector
suineg
13 Jan 2025 00:54
I can also exec into the seat-worker
yeah calendar
I figured it was
Crypta Electrica
13 Jan 2025 00:55
Yeah, so there is no harm in adding seat-discord-connector
or some other driver
If you dont use it just dont configure it
suineg
13 Jan 2025 00:55
how do I add that
docker compose exec front compose required ??
Crypta Electrica
13 Jan 2025 00:56
in your .env file add warlof/seat-discord-connector to your plugin list
suineg
13 Jan 2025 00:56
makes sense, then pull and up
Crypta Electrica
13 Jan 2025 00:56
Yep
No need to pull
But make sure you grab all the docker compose files also
ie
docker compose -f docker-compose.yml -f docker-compose.mariadb.yml -f docker-compose.traefik.yml up -d (if using traefik)
suineg
13 Jan 2025 00:58
ahh I have that aliased to everun so I can everun pull and everun up -d
Crypta Electrica
13 Jan 2025 00:58
Yep that works
Also just looping in @user_171582310783647744
suineg
13 Jan 2025 01:00
awesome, retry cleared it - is Zeno the calendar plugin dev?
Crypta Electrica
13 Jan 2025 01:00
seat-connector
suineg
13 Jan 2025 01:01
I checked the hermesdj/seat-calendar and didn't see anything requiring it so I'm guessing yeah it's downstream
Crypta Electrica
13 Jan 2025 01:01
What is your output of everun exec front composer depends warlof/seat-connector?
suineg
13 Jan 2025 01:02
hermesdj/seat-calendar 1.0.7 requires warlof/seat-connector (^3.0)
warlof/seat-discord-connector 6.0.1 requires warlof/seat-connector (^3.0)
odd
oh that's not odd
Crypta Electrica
13 Jan 2025 01:02
Calendar calls for it
suineg
13 Jan 2025 01:03
yeah it calls for seat-connector but not seat-discord-connector
so it "should" call for that also OR seat-connector should call for minimum one driver?
Crypta Electrica
13 Jan 2025 01:04
Yeah it should probably call for the discord driver too as it expects it to be there
suineg
13 Jan 2025 01:04
or at least "a" driver
Crypta Electrica
13 Jan 2025 01:04
That said it should be perfectly valid to have connector without a driver also
suineg
13 Jan 2025 01:04
if it's slack or discord but something
Crypta Electrica
13 Jan 2025 01:05
And no, calendar is specifically calling for discord. https://github.com/hermesdj/seat-calendar/blob/aaa26e4eebdd3e38b506efee8f0ec91600e133c4/src/Commands/SyncDiscordUsers.php#L50
suineg
13 Jan 2025 01:05
maybe my seat admin went and clicked around and it's looking to tie in
he does have an immediate tutorial on a discord bot
Crypta Electrica
13 Jan 2025 01:06
I mean in this case it goes both ways, calendar will work with only seat-connector installed (even though it will only look for discord users). But seat-connector should also be able to work without a driver
suineg
13 Jan 2025 01:07
all good points from what I can see
Crypta Electrica
13 Jan 2025 01:16
@user_171582310783647744 Feel free to use or do different if you wish https://github.com/zenobio93/seat-connector/pull/1
Zenobio
13 Jan 2025 07:36
I will check on it later, thank you guys