Validate starting item and body type server-side.

This commit is contained in:
Avi Weinstock 2021-03-09 18:25:36 -05:00
parent afbc810b3e
commit f64c29f09d

View File

@ -2,6 +2,15 @@ use crate::persistence::character_loader::CharacterLoader;
use common::comp::{inventory::loadout_builder::LoadoutBuilder, Body, Inventory, Item, Stats};
use specs::{Entity, ReadExpect};
const VALID_STARTER_ITEMS: [&str; 6] = [
"common.items.weapons.hammer.starter_hammer",
"common.items.weapons.bow.starter",
"common.items.weapons.axe.starter_axe",
"common.items.weapons.staff.starter_staff",
"common.items.weapons.sword.starter",
"common.items.weapons.sceptre.starter_sceptre",
];
pub fn create_character(
entity: Entity,
player_uuid: String,
@ -10,13 +19,23 @@ pub fn create_character(
body: Body,
character_loader: &ReadExpect<'_, CharacterLoader>,
) {
// quick fix whitelist validation for now; eventually replace the
// `Option<String>` with an index into a server-provided list of starter
// items, and replace `comp::body::Body` with `comp::body::humanoid::Body`
// throughout the messages involved
let tool_id = match character_tool {
Some(tool_id) if VALID_STARTER_ITEMS.contains(&&*tool_id) => tool_id,
_ => return,
};
if !matches!(body, Body::Humanoid(_)) {
return;
}
let stats = Stats::new(character_alias.to_string());
let loadout = LoadoutBuilder::new()
.defaults()
.active_item(Some(Item::new_from_asset_expect(
character_tool.as_deref().unwrap(),
)))
.active_item(Some(Item::new_from_asset_expect(&tool_id)))
.build();
let mut inventory = Inventory::new_with_loadout(loadout);