diff options
author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-03-27 15:05:47 +0000 |
---|---|---|
committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2009-03-27 15:05:47 +0000 |
commit | e43ce11311017a618db45748d5e3c9a28e1f7b68 (patch) | |
tree | 2260ba626b59085da8b98a97c9070fd56029a66f | |
parent | c3b0ca25b8cd2bc7009d47fb7e9c949701808391 (diff) | |
download | lasso-e43ce11311017a618db45748d5e3c9a28e1f7b68.tar.gz lasso-e43ce11311017a618db45748d5e3c9a28e1f7b68.tar.xz lasso-e43ce11311017a618db45748d5e3c9a28e1f7b68.zip |
XML SAML2: Remove unnecessary overloading of build_query in request/respons objects
* lasso/xml/saml-2.0/samlp2_assertion_id_request.c,
lasso/xml/saml-2.0/samlp2_authn_request.c,
lasso/xml/saml-2.0/samlp2_logout_request.c,
lasso/xml/saml-2.0/samlp2_logout_response.c,
lasso/xml/saml-2.0/samlp2_manage_name_id_request.c,
lasso/xml/saml-2.0/samlp2_manage_name_id_response.c,
lasso/xml/saml-2.0/samlp2_name_id_mapping_request.c,
lasso/xml/saml-2.0/samlp2_name_id_mapping_response.c,
lasso/xml/saml-2.0/samlp2_response.c,
lasso/xml/saml-2.0/samlp2_subject_query_abstract.c: remove useless
overloading of build_query virtual method, revert to implementations
in LassoSamlp2RequestAbstract and LassoSamlp2StatusResponse.
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_assertion_id_request.c | 20 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_authn_request.c | 18 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_logout_request.c | 18 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_logout_response.c | 26 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_manage_name_id_request.c | 18 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_manage_name_id_response.c | 18 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_name_id_mapping_request.c | 18 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_name_id_mapping_response.c | 18 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_response.c | 19 | ||||
-rw-r--r-- | lasso/xml/saml-2.0/samlp2_subject_query_abstract.c | 18 |
10 files changed, 0 insertions, 191 deletions
diff --git a/lasso/xml/saml-2.0/samlp2_assertion_id_request.c b/lasso/xml/saml-2.0/samlp2_assertion_id_request.c index 522da005..bb15de87 100644 --- a/lasso/xml/saml-2.0/samlp2_assertion_id_request.c +++ b/lasso/xml/saml-2.0/samlp2_assertion_id_request.c @@ -49,7 +49,6 @@ /* private methods */ /*****************************************************************************/ - static struct XmlSnippet schema_snippets[] = { { "AssertionIDRef", SNIPPET_CONTENT, G_STRUCT_OFFSET(LassoSamlp2AssertionIDRequest, AssertionIDRef), NULL, NULL, NULL}, @@ -58,23 +57,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLRequest=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -87,7 +69,6 @@ init_from_query(LassoNode *node, char **query_fields) return rc; } - /*****************************************************************************/ /* instance and class init functions */ /*****************************************************************************/ @@ -104,7 +85,6 @@ class_init(LassoSamlp2AssertionIDRequestClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "AssertionIDRequest"); diff --git a/lasso/xml/saml-2.0/samlp2_authn_request.c b/lasso/xml/saml-2.0/samlp2_authn_request.c index cc26337d..4d0ade91 100644 --- a/lasso/xml/saml-2.0/samlp2_authn_request.c +++ b/lasso/xml/saml-2.0/samlp2_authn_request.c @@ -93,23 +93,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLRequest=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -153,7 +136,6 @@ class_init(LassoSamlp2AuthnRequestClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "AuthnRequest"); diff --git a/lasso/xml/saml-2.0/samlp2_logout_request.c b/lasso/xml/saml-2.0/samlp2_logout_request.c index b3c510a3..82e15436 100644 --- a/lasso/xml/saml-2.0/samlp2_logout_request.c +++ b/lasso/xml/saml-2.0/samlp2_logout_request.c @@ -76,23 +76,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLRequest=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -130,7 +113,6 @@ class_init(LassoSamlp2LogoutRequestClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "LogoutRequest"); diff --git a/lasso/xml/saml-2.0/samlp2_logout_response.c b/lasso/xml/saml-2.0/samlp2_logout_response.c index 7a53a219..88a72b98 100644 --- a/lasso/xml/saml-2.0/samlp2_logout_response.c +++ b/lasso/xml/saml-2.0/samlp2_logout_response.c @@ -51,31 +51,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message, *relay_state; - LassoSamlp2LogoutResponse *response = LASSO_SAMLP2_LOGOUT_RESPONSE(node); - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - if (response->relayState) { - relay_state = (char*)xmlURIEscapeStr((xmlChar *)response->relayState, NULL); - ret = g_strdup_printf("SAMLResponse=%s&RelayState=%s", - deflated_message, relay_state); - xmlFree(relay_state); - } else { - ret = g_strdup_printf("SAMLResponse=%s", deflated_message); - } - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -105,7 +80,6 @@ class_init(LassoSamlp2LogoutResponseClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "LogoutResponse"); diff --git a/lasso/xml/saml-2.0/samlp2_manage_name_id_request.c b/lasso/xml/saml-2.0/samlp2_manage_name_id_request.c index 08f55b45..5e5657d3 100644 --- a/lasso/xml/saml-2.0/samlp2_manage_name_id_request.c +++ b/lasso/xml/saml-2.0/samlp2_manage_name_id_request.c @@ -76,23 +76,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLRequest=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -126,7 +109,6 @@ class_init(LassoSamlp2ManageNameIDRequestClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "ManageNameIDRequest"); diff --git a/lasso/xml/saml-2.0/samlp2_manage_name_id_response.c b/lasso/xml/saml-2.0/samlp2_manage_name_id_response.c index c2bef564..6e1c3cc9 100644 --- a/lasso/xml/saml-2.0/samlp2_manage_name_id_response.c +++ b/lasso/xml/saml-2.0/samlp2_manage_name_id_response.c @@ -48,23 +48,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLResponse=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -88,7 +71,6 @@ class_init(LassoSamlp2ManageNameIDResponseClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "ManageNameIDResponse"); diff --git a/lasso/xml/saml-2.0/samlp2_name_id_mapping_request.c b/lasso/xml/saml-2.0/samlp2_name_id_mapping_request.c index 26ac8342..4ea55c69 100644 --- a/lasso/xml/saml-2.0/samlp2_name_id_mapping_request.c +++ b/lasso/xml/saml-2.0/samlp2_name_id_mapping_request.c @@ -70,23 +70,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLRequest=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -119,7 +102,6 @@ class_init(LassoSamlp2NameIDMappingRequestClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "NameIDMappingRequest"); diff --git a/lasso/xml/saml-2.0/samlp2_name_id_mapping_response.c b/lasso/xml/saml-2.0/samlp2_name_id_mapping_response.c index 57f2bbb3..49a647e4 100644 --- a/lasso/xml/saml-2.0/samlp2_name_id_mapping_response.c +++ b/lasso/xml/saml-2.0/samlp2_name_id_mapping_response.c @@ -62,23 +62,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLResponse=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -109,7 +92,6 @@ class_init(LassoSamlp2NameIDMappingResponseClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "NameIDMappingResponse"); diff --git a/lasso/xml/saml-2.0/samlp2_response.c b/lasso/xml/saml-2.0/samlp2_response.c index 4d4d1e1c..62b32491 100644 --- a/lasso/xml/saml-2.0/samlp2_response.c +++ b/lasso/xml/saml-2.0/samlp2_response.c @@ -54,7 +54,6 @@ extern LassoNode* lasso_assertion_encrypt(LassoSaml2Assertion *assertion); /* private methods */ /*****************************************************************************/ - static struct XmlSnippet schema_snippets[] = { { "Assertion", SNIPPET_LIST_NODES, G_STRUCT_OFFSET(LassoSamlp2Response, Assertion), NULL, NULL, NULL}, @@ -65,23 +64,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLResponse=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -153,7 +135,6 @@ class_init(LassoSamlp2ResponseClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->get_xmlNode = get_xmlNode; nclass->node_data = g_new0(LassoNodeClassData, 1); diff --git a/lasso/xml/saml-2.0/samlp2_subject_query_abstract.c b/lasso/xml/saml-2.0/samlp2_subject_query_abstract.c index aa73c5cf..ef025de2 100644 --- a/lasso/xml/saml-2.0/samlp2_subject_query_abstract.c +++ b/lasso/xml/saml-2.0/samlp2_subject_query_abstract.c @@ -58,23 +58,6 @@ static struct XmlSnippet schema_snippets[] = { static LassoNodeClass *parent_class = NULL; - -static gchar* -build_query(LassoNode *node) -{ - char *ret, *deflated_message; - - deflated_message = lasso_node_build_deflated_query(node); - if (deflated_message == NULL) { - return NULL; - } - ret = g_strdup_printf("SAMLRequest=%s", deflated_message); - /* XXX: must support RelayState (which profiles?) */ - g_free(deflated_message); - return ret; -} - - static gboolean init_from_query(LassoNode *node, char **query_fields) { @@ -104,7 +87,6 @@ class_init(LassoSamlp2SubjectQueryAbstractClass *klass) LassoNodeClass *nclass = LASSO_NODE_CLASS(klass); parent_class = g_type_class_peek_parent(klass); - nclass->build_query = build_query; nclass->init_from_query = init_from_query; nclass->node_data = g_new0(LassoNodeClassData, 1); lasso_node_class_set_nodename(nclass, "SubjectQueryAbstract"); |