Azelf reports: Electrumese Sword alt attack ( ... #5346

New Issue

yourland-report · 2023-10-04T00:42:53Z

yourland-report commented

2023-10-04 00:42:53 +00:00

Azelf reports a bug:

Electrumese Sword alt attack (explosive bullets) can destroy shared chests and delete their contents

Player position:

{
	y = -4162.4990234375,
	z = -3058.8000488281,
	x = 157.19900512695
}

Player look:

{
	y = -0.90988886356354,
	z = -0.096493415534496,
	x = 0.40347400307655
}

Player information:

{
	min_rtt = 0.1630000025034,
	avg_rtt = 0.16400000452995,
	min_jitter = 0,
	max_rtt = 8.7910003662109,
	avg_jitter = 0,
	connection_uptime = 1736,
	serialization_version = 29,
	patch = 0,
	formspec_version = 6,
	state = "Active",
	lang_code = "",
	major = 5,
	protocol_version = 42,
	version_string = "5.7.0",
	minor = 7,
	max_jitter = 8.6260004043579,
	ip_version = 6
}

Player meta:

{
	fields = {
		["signslib:pos"] = "(156,-4160,-3059)",
		jointime = "1659363683",
		xp = "1150600",
		["stamina:poisoned"] = "no",
		inflicted_damage = "5340860",
		["stamina:level"] = "20",
		["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 32140\", \"\", \"shields:shield_rainbow 1 33200\", \"3d_armor:leggings_rainbow 1 33200\", \"3d_armor:chestplate_rainbow 1 33200\", \"3d_armor:helmet_rainbow 1 33200\"}",
		yl_church = "return {[\"last_death_portal\"] = 1696340690, [\"last_death\"] = {[\"y\"] = 140, [\"z\"] = 4594, [\"x\"] = -955}, [\"last_heal\"] = 1678197363}",
		["ocean_build.last_warning"] = "1.67602e+09",
		partychat = "party",
		["ocean_build.ocean_built"] = "11",
		["stamina:exhaustion"] = "149",
		arenalib_infobox_arenaID = "0",
		repellant = "0",
		hud_state = "on",
		["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}",
		bitten = "0",
		placed_nodes = "336641",
		played_time = "5929246",
		digged_nodes = "863225",
		yl_commons_thankyou = "93",
		died = "122",
		crafted = "652995",
		punch_count = "251778",
		yl_commons_player_created = "1659363683",
		yl_commons_player_joined = "1696378466"
	}
}

Log identifier


[MOD] yl_report log identifier = IGLxcJIKS2jJMqXL4ObNx984jUM3FcRn

Profiler save:

profile-20231004T004253.json_prettyEE

Status:

# Server: version: 5.7.0-yl-thx-tmm | game: Minetest Game | uptime: 39min | max lag: 0.352s | clients (10/52): AliasAlreadyTaken, Azelf, elelelelel2010, huh, jackofthebean000, Mesprit, MineWorlds, Parrish, Service, shanish

Teleport command:

/teleport xyz 157 -4162 -3059

Compass command:

/give_compass Construction IGLxcJIKS2jJMqXL4ObNx984jUM3FcRn D2691E 157 -4162 -3059

Azelf reports a bug: > Electrumese Sword alt attack (explosive bullets) can destroy shared chests and delete their contents Player position: ``` { y = -4162.4990234375, z = -3058.8000488281, x = 157.19900512695 } ``` Player look: ``` { y = -0.90988886356354, z = -0.096493415534496, x = 0.40347400307655 } ``` Player information: ``` { min_rtt = 0.1630000025034, avg_rtt = 0.16400000452995, min_jitter = 0, max_rtt = 8.7910003662109, avg_jitter = 0, connection_uptime = 1736, serialization_version = 29, patch = 0, formspec_version = 6, state = "Active", lang_code = "", major = 5, protocol_version = 42, version_string = "5.7.0", minor = 7, max_jitter = 8.6260004043579, ip_version = 6 } ``` Player meta: ``` { fields = { ["signslib:pos"] = "(156,-4160,-3059)", jointime = "1659363683", xp = "1150600", ["stamina:poisoned"] = "no", inflicted_damage = "5340860", ["stamina:level"] = "20", ["3d_armor_inventory"] = "return {\"nether_mobs:dragon_boots 1 32140\", \"\", \"shields:shield_rainbow 1 33200\", \"3d_armor:leggings_rainbow 1 33200\", \"3d_armor:chestplate_rainbow 1 33200\", \"3d_armor:helmet_rainbow 1 33200\"}", yl_church = "return {[\"last_death_portal\"] = 1696340690, [\"last_death\"] = {[\"y\"] = 140, [\"z\"] = 4594, [\"x\"] = -955}, [\"last_heal\"] = 1678197363}", ["ocean_build.last_warning"] = "1.67602e+09", partychat = "party", ["ocean_build.ocean_built"] = "11", ["stamina:exhaustion"] = "149", arenalib_infobox_arenaID = "0", repellant = "0", hud_state = "on", ["unified_inventory:bags"] = "return {\"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\", \"unified_inventory:bag_large\"}", bitten = "0", placed_nodes = "336641", played_time = "5929246", digged_nodes = "863225", yl_commons_thankyou = "93", died = "122", crafted = "652995", punch_count = "251778", yl_commons_player_created = "1659363683", yl_commons_player_joined = "1696378466" } } ``` Log identifier ``` [MOD] yl_report log identifier = IGLxcJIKS2jJMqXL4ObNx984jUM3FcRn ``` Profiler save: ``` profile-20231004T004253.json_prettyEE ``` Status: ``` # Server: version: 5.7.0-yl-thx-tmm | game: Minetest Game | uptime: 39min | max lag: 0.352s | clients (10/52): AliasAlreadyTaken, Azelf, elelelelel2010, huh, jackofthebean000, Mesprit, MineWorlds, Parrish, Service, shanish ``` Teleport command: ``` /teleport xyz 157 -4162 -3059 ``` Compass command: ``` /give_compass Construction IGLxcJIKS2jJMqXL4ObNx984jUM3FcRn D2691E 157 -4162 -3059 ```

AliasAlreadyTaken was assigned by yourland-report

2023-10-04 00:42:53 +00:00

AzelfYL commented

2023-10-04 00:47:23 +00:00

I accidentally used my electrumese sword's alternate attack on some of the chests while trying to place it into one of the chests, and the attack destroyed two chests in my base and deleted their contents (one was a chest of raw meat and the other one contained spare gear). I checked, and they did not enter the TP tube because there's none of the gear at the other end and testing on the test server proved that the contents do indeed get deleted.

screenshot_20231004_084323.png

380 KiB

DragonWrangler commented

2023-10-04 02:49:04 +00:00

It's always annoying when that happens

AzelfYL commented

2023-10-14 01:23:45 +00:00

tag when lol

flux added the

1. kind/bug

3. source/integration

labels 2023-11-06 21:30:54 +00:00

flux added this to the flux's TODO list project 2023-11-06 21:31:33 +00:00

flux referenced this issue from a commit

2023-11-06 21:42:53 +00:00

your-land/bugtracker#5346 prevent chesttools chests from being destroyed by explosion and losing thier contents

flux commented

2023-11-06 21:43:15 +00:00

i need to get back to the color-able shared chest update at some point.

in the meantime, fixed in integration: 311a8f4dcf

i need to get back to the color-able shared chest update at some point. in the meantime, fixed in integration: https://gitea.your-land.de/your-land/yl_commons/commit/311a8f4dcf486fd88062342bbac1633f2650739e

flux added the

4. step/ready to QA test

label 2023-11-06 21:43:23 +00:00

AliasAlreadyTaken added this to the 1.1.122 milestone 2023-11-11 20:25:32 +00:00

flux commented

2023-11-17 18:05:17 +00:00

technically this is a dupe of #4406, though that one's blocked on an upstream PR whereas this one got fixed in integration

AliasAlreadyTaken commented

2023-11-27 00:28:54 +00:00

Positions

156,-4162,-3059
155,-4162,-3059

Those chests are not in the backup, they changed too often ever since. But there's the log.

Positions 156,-4162,-3059 155,-4162,-3059 Those chests are not in the backup, they changed too often ever since. But there's the log.

AliasAlreadyTaken commented

2023-12-03 22:06:16 +00:00

We'll refund the chests.

QA

With chests not exploding anymore, it won't take long until people build themselves a nether base out of those. We probably need to make up our mind what should happen in such a case. However, the chest doesn't explode anymore.

We'll refund the chests. QA With chests not exploding anymore, it won't take long until people build themselves a nether base out of those. We probably need to make up our mind what should happen in such a case. However, the chest doesn't explode anymore.

AliasAlreadyTaken added the

ugh/QA OK

label 2023-12-03 22:06:24 +00:00

DragonWrangler commented

2023-12-03 22:07:58 +00:00

why not make it where chests can't be placed in the nether ?

AliasAlreadyTaken commented

2023-12-03 22:14:10 +00:00

why not make it where chests can't be placed in the nether ?

Because there's no reason to disallow placing chests in the nether. Plus, it doesn't solve the basic issue

> why not make it where chests can't be placed in the nether ? Because there's no reason to disallow placing chests in the nether. Plus, it doesn't solve the basic issue

flux commented

2023-12-03 23:10:01 +00:00

there's lots of other things that already don't explode, and e.g. locked chests take only 1/2 the iron that the shared chest does, so this doesn't open up any new exploits.

👍 1

flux added

5. result/fixed

and removed

4. step/ready to QA test

labels 2023-12-18 00:38:51 +00:00

flux removed this from the flux's TODO list project 2023-12-18 00:38:53 +00:00

AliasAlreadyTaken was unassigned by flux

2023-12-18 00:38:56 +00:00

flux commented

2023-12-18 00:39:04 +00:00

this is live