Skip to content

Address exploit of container dump module #1550

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

Open
RedRafe wants to merge 5 commits into
base: develop
Choose a base branch
from
+145 −69
Open

Address exploit of container dump module #1550

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
4f18947
Address exploit of container dump module
RedRafe Feb 20, 2026
5e92acf
Update market chest exchange values
RedRafe Feb 20, 2026
4cebffc
Replace kill mechanic with stun mechanic
RedRafe Feb 23, 2026
a717dae
Separate vehicle penalty logic in configs
RedRafe Feb 23, 2026
e85d7be
Change default config
RedRafe Feb 23, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 5 additions & 5 deletions config.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -590,11 +590,11 @@ storage.config = {
market_provides_chests = true,
-- What market provides
offers = {
['coal'] = 2,
['copper-ore'] = 2,
['iron-ore'] = 2,
['stone'] = 2,
['uranium-ore'] = 10,
['coal'] = 5,
['copper-ore'] = 5,
['iron-ore'] = 5,
['stone'] = 5,
['uranium-ore'] = 25,
},
-- What market requests
requests = {
Expand Down
2 changes: 1 addition & 1 deletion features/gui/experience.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -483,7 +483,7 @@ local function on_entity_died(event)

-- stuff killed by the player force, but not the player
if force and force.name == 'player' then
if cause and (cause.name == 'artillery-turret' or cause.name == 'gun-turret' or cause.name == 'laser-turret' or cause.name == 'flamethrower-turret') then
if cause and cause.valid and (cause.name == 'artillery-turret' or cause.name == 'gun-turret' or cause.name == 'laser-turret' or cause.name == 'flamethrower-turret') then
exp = config.XP['enemy_killed'] * (config.alien_experience_modifiers[entity_name] or 1)
floating_text_position = cause.position
else
Expand Down
44 changes: 44 additions & 0 deletions map_gen/maps/danger_ores/modules/container_dump.lua
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,31 @@ Global.register(

local Public = {}

local cause_by_type = {
['character'] = function(cause)
return cause.player
end,
['car'] = function(cause)
local d = cause.get_driver()
if d then
return (d.object_name == 'LuaEntity') and d.player or d
else
return cause.last_user
end
end,
['spider-vehicle'] = function(cause)
local d = cause.get_driver()
if d then
return (d.object_name == 'LuaEntity') and d.player or d
else
return cause.last_user
end
end,
['land-mine'] = function(cause)
return cause.last_user
end,
}

Public.register = function(config)
local entity_name = config.entity_name or 'coal'
Event.add(
Expand Down Expand Up @@ -76,6 +101,25 @@ Public.register = function(config)
end

died_entities[entity.unit_number] = true

local cause = event.cause
if not (cause and cause.valid and cause.force and cause.force.name == 'player') then
return
end

local handler = cause_by_type[cause.type]
local actor = handler and handler(cause)
if not (actor and actor.valid) then
return
end

local character = actor.character
if not (character and character.valid) then
return
end

actor.print('The ore fights back!', { color = { 255, 128, 0 } })
character.die(game.forces.neutral)
Copy link
Collaborator

@grilledham grilledham Feb 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder if the dying part should be configurable. This seems more like a diggy danger ore issue than a regular danger ore issue. Part of me feels that killing the player is quite harsh, but the only alternative I can think of is spilling the items instead. Though I suppose it's odd for players to accidentally destroy containers, outside of using explosives to kill biters and the container getting caught in the crossfire.

Copy link
Contributor Author

@RedRafe RedRafe Feb 20, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Experience of the last X months is: in regular DO almost no-one ever kills a container (mainly because ppl know it would spawn "useless" coal to mine, and also because there's no real need to get rid of stuff to make room). So feature is pretty much irrelevant.

For Diggy, the infinite resource loop is a more relevant concern. We had in DO the powered-warehouses, and they would start spilling content on ground when left unpowered. That made everyone angry at the mechanic cuz it punished the whole party rather than the single griefer/offender.

IMO, neither killing nor spawning ore on a 1x1 or NxN area is an harsh punishment, as respawning is a matter of 10s and mining out the ore tiles is more an annoyance than a real issue.

It only acts as a "warning" after 1st attempt, since it wastes a bit of player time and cannot be automated, players wont do it again ever after.

I dont think this feature is triggered enough to cause any issues among regular players. Maybe griefers would be the only ones experiencing a few deaths from time to time, but it's still unusual to get kills in DO or to see players blow up chest at all. A game can easily finish with 0 deaths total (no trains, peaceful,...).

If so, we can still revert the change again and turn it down a notch.

Copy link
Collaborator

@grilledham grilledham Feb 21, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Experience of the last X months is: in regular DO almost no-one ever kills a container (mainly because ppl know it would spawn "useless" coal to mine, and also because there's no real need to get rid of stuff to make room). So feature is pretty much irrelevant.

As in the feature works in it's current form and doesn't need changing for DO?

IMO, neither killing nor spawning ore on a 1x1 or NxN area is an harsh punishment, as respawning is a matter of 10s and mining out the ore tiles is more an annoyance than a real issue.

For large maps it can be more than a 10s inconvenience as it's also the time to get back to your corpse. Plus it would interrupt what you are doing if you are in the middle of building something while clearing chests. Maybe I am over reacting about killing the player. But it sets the tone for the map, particularly for new players. I could imagine a new player to DO would naturally try to destroy containers to take advantage of the DO game mechanics. Staying alive makes it easier to see/notice the spawned ore. I'd be more accepting of killing players in diggy as that is inherently a more dangerous map (DO normally has bitters to peaceful).

Though thinking about this again, there are some other solutions we could explore.

  1. Don't use the container module for diggy DO. The mechanic of the map are different for diggy around clearing space and resource scarcity. So maybe this feature just isn't needed.
  2. For diggy DO lower the amount of coal spawned to make the exploit not viable. Again as the mechanics for diggy are different a fraction of the ore spawn is still inconvenient without being able to exploit freeing space as that happens mostly through mining rocks.
  3. Change the market ratio to make the expolit not viable.
  4. Combination of 2 and 3.

Copy link
Contributor Author

@RedRafe RedRafe Feb 22, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For once, I find myself in disagreement with the concerns raised.

I'll try to explain the issue so a better solution can be found:

  1. Fo regular DO, this feature in any of it's implementations has little to no impact as no chests are ever killed in a DO run. Neither to make space, nor to exchange resources to coal, nor by accident.

  2. For DiggyDO, Diggy used a separate DO logic, but for the sake of reusing and maintaining, it was switched to use DO's files & implementation around when both scenarios got heavily refractored at the end of 1.1 and 2.0 porting.

The issue with Diggy is that long lasting maps generate absurd resource imbalance and players started to blow up chests of whatever was overflowing to keep the base running. To address this mechanic, 3 features were added back in few versions ago:

  • disabling mining prod (from DO)
  • container dump (from DO)
  • market chest (new diggy feature)

The combination of those 3 was meant to incentivize players into using all resources as best as possible; and then if really needed, trade excess at bad ratios instead of voiding it in chests (instead of storing 420M ores in bot network).

Players later realized that without mining prod, diggy needed a lot of expansion and manual work to feed the factory. So the exploit to kill chest of cables, mine coal, exchange for copper &brevraft in cable loop was found to generate infinite stream of resources.

For latest Diggy version, exchange rates have been raised from 2:1 to 5:1 to offset the prod3s chain already - however - since constantly expanding manually was deemed too tedious for large SPM goals - mining prod has been introduced back. Now no value of exchange rate can offset the productivity gained from all the steps involved. So I nerfed the infinite loop by basically debouncing the loop by killing poster when it destroys a chest.

Lore-wise, I personally found it very fitting with the danger of ores theme, and penalty was not received as too harsh neither in discussion with players nor in tests with public server.

However, if a better solution can be found, I'm open to it. I haven't quite found a fix to address all these issues so far

Copy link
Collaborator

@grilledham grilledham Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Are you saying the container dump module isn't needed for regular DO, or that this change to the module will have no impact on DO runs?

Either way I think we will just have to agree to disagree. This doesn't feel like the right way to solve the problem to me. But I'm open to trying it and seeing what players say.

Copy link
Contributor Author

@RedRafe RedRafe Feb 22, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Are you saying the container dump module isn't needed for regular DO, or that this change to the module will have no impact on DO runs?

Both - DO doesnt need container dump as antigrief/anticheat system as players already behave for that matter, but I find it very fitting to the lore/design of the map so I wouldn't necessarily remove it even if it's almost unused.
Given that it is not a relevant feature, changes to it (like harsh penalty of character death) would not have relevant impact as feature is not triggered enough for players to notice.

Even tho we might be in disagreement I dont want to push changes that are not shared. I dont need this to go through as is just for the sake of it.
A new Diggy run has already restarted and it is testing these changes + some some other little tweaks, so I'm in no rush :)

If we think death is too harsh a penalty, we can maybe just stun the player. Looking at base protos, there's landmines, discharge defense and (with SA) tesla turrets that have this "stun mechanic":

action = {
    type = "create-sticker",
    sticker = "stun-sticker"
}

(A sticker is some kind of entity - enemy green spits are "sticker" that get attached to player and fade with time just to give context).
Maybe I can fiddle around with that and see if I can attach stun stickers around target actors (will have to check how it works with vehicles but spit do attach to tanks/cars so it should be possible). This would be sufficient I believe as debouncing mechanism to prevent players from running over a row of chests and convert it to coal for endless mining.

Bonus: if it's not enough to stop it, then I can additionally kill just the vehicle and maybe also spawn some enemies as well as coal so the feature is less predictable from player side

Copy link
Collaborator

@grilledham grilledham Feb 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both - DO doesnt need container dump as antigrief/anticheat system as players already behave for that matter

I don't think this is correct. I added the module originally because we did have a problem with this.

Even tho we might be in disagreement I dont want to push changes that are not shared.

I appreciate that 😊

If we think death is too harsh a penalty, we can maybe just stun the player.

I would prefer stunning the player over killing them, so if you can make that work, that works for me.

Copy link
Contributor Author

@RedRafe RedRafe Feb 22, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added the module originally because we did have a problem with this.

Sure! And I think nowadays the combination of regulars knowing there's a penalty to killing chests of ore and them understanding game economy a Lil better has lead to no chests being blown up anymore.

DO is very chill so the only "precious" resource is time. Extracting resources takes time an nobody wants to mine twice a tile because someone blew a chest. Even if not triggered anymore, I think if the feature was disabled ppl would start blowing up chest again

end
)

Expand Down