query-planner: avoid `panic!` when parsing Federation directives

Right now, we `panic!` in multiple places. This should be avoided and we should print a better error that explains the rejection of the supergraph due to invalid values.